{"push_id":1319240882,"size":27,"distinct_size":11,"ref":"refs/heads/KAFKA-4177","head":"5bbf9a1b5a812e24f56b81ab70c1b6175e440e6e","before":"54d4a8cf59a1739a9c9b7cbb6b0c35db64309642","commits":[{"sha":"24f81ea764a493b4422b6a3ef6b3e771d0e4d63b","author":{"name":"Damian Guy","email":"a352d70b5750eea9e7969230d96f9ec481f91a1c@gmail.com"},"message":"MINOR: add javadoc comment to PersistenKeyValueFactory.enableCaching\n\nmissing javadoc on public API method PersistenKeyValueFactory.enableCaching\n\nAuthor: Damian Guy <damian.guy@gmail.com>\n\nReviewers: Eno Thereska, Guozhang Wang\n\nCloses #1891 from dguy/minor-java-doc","distinct":false,"url":"https://api.github.com/repos/benstopford/kafka/commits/24f81ea764a493b4422b6a3ef6b3e771d0e4d63b"},{"sha":"a632716a3c9a871f325c6f13aefa9aed0add4b82","author":{"name":"Damian Guy","email":"a352d70b5750eea9e7969230d96f9ec481f91a1c@gmail.com"},"message":"MINOR: remove unused code from InternalTopicManager\n\nRemove isValidCleanupPolicy and related fields as they are never used.\n\nAuthor: Damian Guy <damian.guy@gmail.com>\n\nReviewers: Eno Thereska, Guozhang Wang\n\nCloses #1888 from dguy/minor-remove-unused","distinct":false,"url":"https://api.github.com/repos/benstopford/kafka/commits/a632716a3c9a871f325c6f13aefa9aed0add4b82"},{"sha":"732fabf94ebc9631d31f2feb2116ee8b63beabef","author":{"name":"Jason Gustafson","email":"158e602a35675d2a0b685478824aa5995557a235@confluent.io"},"message":"KAFKA-3782: Ensure heartbeat thread restarted after rebalance woken up\n\nAuthor: Jason Gustafson <jason@confluent.io>\n\nReviewers: Guozhang Wang\n\nCloses #1898 from hachikuji/KAFKA-3782","distinct":false,"url":"https://api.github.com/repos/benstopford/kafka/commits/732fabf94ebc9631d31f2feb2116ee8b63beabef"},{"sha":"27e3edc791760dea7ff4d048f87d1585f9e235d7","author":{"name":"Elias Levy","email":"c4facf758d977fc84f57189af11b64823a45b7e8@gmail.com"},"message":"MINOR: Fix comments in KStreamKStreamJoinTest\n\nMinor comment fixes.\n\nAuthor: Elias Levy <fearsome.lucidity@gmail.com>\n\nReviewers: Guozhang Wang\n\nCloses #1885 from eliaslevy/fix-test-comments","distinct":false,"url":"https://api.github.com/repos/benstopford/kafka/commits/27e3edc791760dea7ff4d048f87d1585f9e235d7"},{"sha":"4112bc3af7bec63ae1248c1b9db745ba8ac4c63c","author":{"name":"Eno Thereska","email":"558d1204cdfaa2922fe8df88ce1f18639c19ca70@gmail.com"},"message":"HOTFIX: Decrease commit interval\n\nThe original commit interval of 30 seconds might be too large in some cases, e.g., when the verifier finishes before those 30 seconds have elapsed.\n\nAuthor: Eno Thereska <eno.thereska@gmail.com>\n\nReviewers: Damian Guy, Guozhang Wang\n\nCloses #1899 from enothereska/hotfix-smoke-test-commit-interval","distinct":false,"url":"https://api.github.com/repos/benstopford/kafka/commits/4112bc3af7bec63ae1248c1b9db745ba8ac4c63c"},{"sha":"d32f3f2828c199a5a09bbba779d07430e43fb190","author":{"name":"Ismael Juma","email":"e6abb59dc9dfc569a27afac0f1a05d72488c0302@juma.me.uk"},"message":"MINOR: Increase `zkConnectionTimeout` and timeout in `testReachableServer`\n\nWe had a number of failures recently due to these timeouts being too low. It's a particular problem if multiple forks are used while running the tests.\n\nAuthor: Ismael Juma <ismael@juma.me.uk>\n\nReviewers: Jason Gustafson <jason@confluent.io>\n\nCloses #1889 from ijuma/increase-zk-timeout-in-tests","distinct":false,"url":"https://api.github.com/repos/benstopford/kafka/commits/d32f3f2828c199a5a09bbba779d07430e43fb190"},{"sha":"106a7456060750ab0604d290b8c1e33a57adbf20","author":{"name":"Jason Gustafson","email":"158e602a35675d2a0b685478824aa5995557a235@confluent.io"},"message":"MINOR: Add test cases for delays in consumer rebalance listener\n\nAuthor: Jason Gustafson <jason@confluent.io>\n\nReviewers: Ismael Juma <ismael@juma.me.uk>\n\nCloses #1866 from hachikuji/rebalance-delay-test-cases","distinct":false,"url":"https://api.github.com/repos/benstopford/kafka/commits/106a7456060750ab0604d290b8c1e33a57adbf20"},{"sha":"6a13a3dbaddf99850b2583007577fa2a6e1e6d3a","author":{"name":"Jason Gustafson","email":"158e602a35675d2a0b685478824aa5995557a235@confluent.io"},"message":"KAFKA-3590; Handle not-enough-replicas errors when writing to offsets topic\n\nAuthor: Jason Gustafson <jason@confluent.io>\n\nReviewers: Ismael Juma <ismael@juma.me.uk>, Guozhang Wang <wangguoz@gmail.com>\n\nCloses #1859 from hachikuji/KAFKA-3590","distinct":false,"url":"https://api.github.com/repos/benstopford/kafka/commits/6a13a3dbaddf99850b2583007577fa2a6e1e6d3a"},{"sha":"0de807357be4d776747e960b5fd17ddeae9a6cad","author":{"name":"Ryan Pridgeon","email":"a36e4e75112566d0718b8c34dcbf7c84ea43c79f@gmail.com"},"message":"KAFKA-3719; Allow underscores in hostname\n\nTechnically this does not strictly adhere to RFC-952 however it is valid for domain names, urls and uris so we should loosen the requirements a tad.\n\nAuthor: Ryan Pridgeon <ryan.n.pridgeon@gmail.com>\n\nReviewers: Ismael Juma <ismael@juma.me.uk>\n\nCloses #1856 from rnpridgeon/KAFKA-3719","distinct":false,"url":"https://api.github.com/repos/benstopford/kafka/commits/0de807357be4d776747e960b5fd17ddeae9a6cad"},{"sha":"36242b846a42b33d7d4c1931f2dae93ebe1547c7","author":{"name":"Sumit Arrawatia","email":"84657557e90e71afed20aedcf9f120417ad2f22d@gmail.com"},"message":"KAFKA-4151; Update public docs for Cluster Id (KIP-78)\n\n- Updated implementation docs with details on Cluster Id generation.\n- Mention cluster id in \"noteworthy changes for 0.10.1.0\" in upgrade docs.\n\nAuthor: Sumit Arrawatia <sumit.arrawatia@gmail.com>\nAuthor: arrawatia <sumit.arrawatia@gmail.com>\n\nReviewers: Ismael Juma <ismael@juma.me.uk>\n\nCloses #1895 from arrawatia/kip-78-docs","distinct":false,"url":"https://api.github.com/repos/benstopford/kafka/commits/36242b846a42b33d7d4c1931f2dae93ebe1547c7"},{"sha":"1d055f7551d138324d2540095a1cfc1c8f74d76f","author":{"name":"Arun Mahadevan","email":"23499836efe5be9cbca864f41f4550b75f910aa9@hortonworks.com"},"message":"KAFKA-3282; Change tools to use new consumer if zookeeper is not specified\n\nAuthor: Arun Mahadevan <aiyer@hortonworks.com>\n\nReviewers: Jason Gustafson <jason@confluent.io>, Ismael Juma <ismael@juma.me.uk>\n\nCloses #1376 from arunmahadevan/cons-consumer-fix","distinct":false,"url":"https://api.github.com/repos/benstopford/kafka/commits/1d055f7551d138324d2540095a1cfc1c8f74d76f"},{"sha":"c0a62b70a8eadc550c937bb18e0203ab691618f5","author":{"name":"Rajini Sivaram","email":"845649f006983e12b0a3bbde4be398021e14793f@googlemail.com"},"message":"KAFKA-4055; System tests for secure quotas\n\nFix existing client-id quota test which currently don't configure quota overrides correctly. Add new tests for user and (user, client-id) quota overrides and default quotas.\n\nAuthor: Rajini Sivaram <rajinisivaram@googlemail.com>\n\nReviewers: Jun Rao <junrao@gmail.com>\n\nCloses #1860 from rajinisivaram/KAFKA-4055","distinct":false,"url":"https://api.github.com/repos/benstopford/kafka/commits/c0a62b70a8eadc550c937bb18e0203ab691618f5"},{"sha":"b75245cfbbefc712103b9329da0f27a205baa6aa","author":{"name":"Jason Gustafson","email":"158e602a35675d2a0b685478824aa5995557a235@confluent.io"},"message":"MINOR: Wakeups propagated from commitOffsets in WorkerSinkTask should be caught\n\nAuthor: Jason Gustafson <jason@confluent.io>\n\nReviewers: Ewen Cheslack-Postava <ewen@confluent.io>\n\nCloses #1907 from hachikuji/catch-wakeup-worker-sink-task","distinct":false,"url":"https://api.github.com/repos/benstopford/kafka/commits/b75245cfbbefc712103b9329da0f27a205baa6aa"},{"sha":"be6056abc9970600347c95c4c8658799b76dbe6b","author":{"name":"Apurva Mehta","email":"d3bf0f90467e0dd83508c95367a21ab0f97f472a@gmail.com"},"message":"KAFKA-4214; kafka-reassign-partitions fails all the time when brokers are bounced during reassignment\n\nThere is a corner case bug, where during partition reassignment, if the\ncontroller and a broker receiving a new replica are bounced at the same\ntime, the partition reassignment is failed.\n\nThe cause of this bug is a block of code in the KafkaController which\nfails the reassignment if the aliveNewReplicas != newReplicas, ie. if\nsome of the new replicas are offline at the time a controller fails\nover.\n\nThe fix is to have the controller listen for ISR change events even for\nnew replicas which are not alive when the controller boots up. Once the\nsaid replicas come online, they will be in the ISR set, and the new\ncontroller will detect this, and then mark the reassignment as\nsuccessful.\n\nInterestingly, the block of code in question was introduced in\nKAFKA-990, where a concern about this exact scenario was raised :)\n\nThis bug was revealed in the system tests in https://github.com/apache/kafka/pull/1904.\nThe relevant tests will be enabled in either this or a followup PR when PR-1904 is merged.\n\nThanks to junrao identifying the issue and providing the patch.\n\nAuthor: Apurva Mehta <apurva.1618@gmail.com>\n\nReviewers: Jun Rao <junrao@gmail.com>\n\nCloses #1910 from apurvam/KAFKA-4214","distinct":false,"url":"https://api.github.com/repos/benstopford/kafka/commits/be6056abc9970600347c95c4c8658799b76dbe6b"},{"sha":"60ad6d727861a87fa756918a7be7547e9b1f4c3d","author":{"name":"Vahid Hashemian","email":"3cbd77b9f48a475f3d76cb73e4791ace5a0d82d4@us.ibm.com"},"message":"KAFKA-3831; Prepare for updating new-consumer-based Mirror Maker's default partition assignment strategy to round robin\n\nThis patch adds proper warning message and necessary doc updates for updating the default partition assignment strategy of Mirror Maker from range to round robin. The actual switch would occur as part of a major release cycle (to be scheduled).\n\nAuthor: Vahid Hashemian <vahidhashemian@us.ibm.com>\n\nReviewers: Jason Gustafson <jason@confluent.io>\n\nCloses #1499 from vahidhashemian/KAFKA-3831","distinct":false,"url":"https://api.github.com/repos/benstopford/kafka/commits/60ad6d727861a87fa756918a7be7547e9b1f4c3d"},{"sha":"a208478f37f216324a87b530f18f30ecd79601c8","author":{"name":"Magnus Reftel","email":"ef663ffd4dbaad20013aa7715c46789b3b2fc701@skatteetaten.no"},"message":"MINOR: Fix Javadoc of @throws AuthorizationException for KafkaConsumer.poll\n\nAuthor: Magnus Reftel <magnus.reftel@skatteetaten.no>\n\nReviewers: Jason Gustafson <jason@confluent.io>\n\nCloses #1901 from reftel/feature/poll_javadoc","distinct":false,"url":"https://api.github.com/repos/benstopford/kafka/commits/a208478f37f216324a87b530f18f30ecd79601c8"},{"sha":"a398fd7d8f60c2f8ba8c0c4d7cac12754e94eea1","author":{"name":"Ben Stopford","email":"c0d82e1daf9ac494499d27d4fb35099385745d7f@gmail.com"},"message":"KAFKA-4177: Refactor to pull out the dynamic broker property replication.quota.throttled.rate from KafkaConfig and into its own class collocated with the ConfigCommand/AdminUtils","distinct":true,"url":"https://api.github.com/repos/benstopford/kafka/commits/a398fd7d8f60c2f8ba8c0c4d7cac12754e94eea1"},{"sha":"808baff362295ffc29f8c810212d9ee7c298dfe7","author":{"name":"Ben Stopford","email":"c0d82e1daf9ac494499d27d4fb35099385745d7f@gmail.com"},"message":"KAFKA-4177: Removed ClientQuotasTest which was accidentally added back in with merge","distinct":true,"url":"https://api.github.com/repos/benstopford/kafka/commits/808baff362295ffc29f8c810212d9ee7c298dfe7"},{"sha":"a23e122de2c743219bcbeeb06b09c3c94feb974a","author":{"name":"Ben Stopford","email":"c0d82e1daf9ac494499d27d4fb35099385745d7f@gmail.com"},"message":"KAFKA-4177: Added 'user' validation to AdminUtils which matches client validation","distinct":true,"url":"https://api.github.com/repos/benstopford/kafka/commits/a23e122de2c743219bcbeeb06b09c3c94feb974a"},{"sha":"984a12ad2bf01392b35e88581c217b10ff2426ae","author":{"name":"Ben Stopford","email":"c0d82e1daf9ac494499d27d4fb35099385745d7f@gmail.com"},"message":"KAFKA-4177: Remenance of merge conflict","distinct":true,"url":"https://api.github.com/repos/benstopford/kafka/commits/984a12ad2bf01392b35e88581c217b10ff2426ae"}]}
{"actor":{"display_login":"benstopford"}}