[GitHub] kafka pull request #3554: KAFKA-5123 Refactor ZkUtils readData* methods

2017-12-14 Thread baluchicken
Github user baluchicken closed the pull request at: https://github.com/apache/kafka/pull/3554 ---

[GitHub] kafka pull request #3637: KAFKA-4879 KafkaConsumer.position may hang forever...

2017-08-07 Thread baluchicken
GitHub user baluchicken opened a pull request: https://github.com/apache/kafka/pull/3637 KAFKA-4879 KafkaConsumer.position may hang forever when deleting a topic @guozhangwang I created this PR but it lacks the test. I also has the test but I wanted to ask which Test file should

[GitHub] kafka pull request #3554: KAFKA-5123 Refactor ZkUtils readData* methods

2017-07-20 Thread baluchicken
GitHub user baluchicken opened a pull request: https://github.com/apache/kafka/pull/3554 KAFKA-5123 Refactor ZkUtils readData* methods You can merge this pull request into a Git repository by running: $ git pull https://github.com/baluchicken/kafka-1 KAFKA-5123 Alternatively

[GitHub] kafka pull request #3281: KAFKA-5388 Replace zkClient.subscribe*Changes meth...

2017-06-09 Thread baluchicken
GitHub user baluchicken opened a pull request: https://github.com/apache/kafka/pull/3281 KAFKA-5388 Replace zkClient.subscribe*Changes method with an equivalent zkUtils method @ijuma can you please review if you have time:) ? You can merge this pull request into a Git repository

[GitHub] kafka pull request #3245: KAFKA-5391 Replace zkClient.delete* method with an...

2017-06-06 Thread baluchicken
GitHub user baluchicken opened a pull request: https://github.com/apache/kafka/pull/3245 KAFKA-5391 Replace zkClient.delete* method with an equivalent zkUtills @ijuma can you please review. You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] kafka pull request #3243: KAFKA-5389 Replace zkClient.exists method with zkU...

2017-06-06 Thread baluchicken
GitHub user baluchicken opened a pull request: https://github.com/apache/kafka/pull/3243 KAFKA-5389 Replace zkClient.exists method with zkUtils.pathExists @ijuma can you please review You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] kafka pull request #3097: KAFKA-5134 Replace zkClient.getChildren method wit...

2017-05-19 Thread baluchicken
GitHub user baluchicken opened a pull request: https://github.com/apache/kafka/pull/3097 KAFKA-5134 Replace zkClient.getChildren method with zkUtils.getChildren @ijuma plz review. I also created a version where I didn't refactor the getPartitionAssignmentForTopics

[GitHub] kafka pull request #2981: KAFKA-5173: Add logging to dump JaasContext before...

2017-05-08 Thread baluchicken
Github user baluchicken closed the pull request at: https://github.com/apache/kafka/pull/2981 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] kafka pull request #2981: KAFKA-5173: Add logging to dump JaasContext before...

2017-05-05 Thread baluchicken
GitHub user baluchicken opened a pull request: https://github.com/apache/kafka/pull/2981 KAFKA-5173: Add logging to dump JaasContext before writing it to file @rajinisivaram plz review. You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] kafka pull request #2919: KAFKA-5127 Replace pattern matching with foreach w...

2017-04-26 Thread baluchicken
GitHub user baluchicken opened a pull request: https://github.com/apache/kafka/pull/2919 KAFKA-5127 Replace pattern matching with foreach where the case None … @ijuma plz review. This one is not complete because KafkaController, AdminUtils and ReplicaStateMachine has some

[GitHub] kafka pull request #2888: KAFKA-5103 Refactor AdminUtils to use zkUtils meth...

2017-04-21 Thread baluchicken
GitHub user baluchicken opened a pull request: https://github.com/apache/kafka/pull/2888 KAFKA-5103 Refactor AdminUtils to use zkUtils methods instad of zkUti… @ijuma plz review. You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] kafka pull request #2886: KAFKA-5101 Remove KafkaController's incrementContr...

2017-04-21 Thread baluchicken
GitHub user baluchicken opened a pull request: https://github.com/apache/kafka/pull/2886 KAFKA-5101 Remove KafkaController's incrementControllerEpoch method p… @ijuma minor fix. can you please review. You can merge this pull request into a Git repository by running: $ git

[GitHub] kafka pull request #2732: KAFKA-4855 Struct SchemaBuilder should not allow d...

2017-03-24 Thread baluchicken
GitHub user baluchicken opened a pull request: https://github.com/apache/kafka/pull/2732 KAFKA-4855 Struct SchemaBuilder should not allow duplicate fields @ewencp can you please review. You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] kafka pull request #2506: KAFKA-4703 Test with two SASL_SSL listeners with d...

2017-02-06 Thread baluchicken
GitHub user baluchicken opened a pull request: https://github.com/apache/kafka/pull/2506 KAFKA-4703 Test with two SASL_SSL listeners with different JAAS contexts @ijuma plz review You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] kafka pull request #2423: KAFKA-4679 Remove unstable markers from Connect AP...

2017-01-23 Thread baluchicken
GitHub user baluchicken opened a pull request: https://github.com/apache/kafka/pull/2423 KAFKA-4679 Remove unstable markers from Connect APIs @ewencp ignore this PR if you are already started to work on this ticket. You can merge this pull request into a Git repository by running

[GitHub] kafka pull request #2422: KAFKA-4679 Remove unstable markers from Connect AP...

2017-01-23 Thread baluchicken
Github user baluchicken closed the pull request at: https://github.com/apache/kafka/pull/2422 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] kafka pull request #2422: KAFKA-4679 Remove unstable markers from Connect AP...

2017-01-23 Thread baluchicken
GitHub user baluchicken opened a pull request: https://github.com/apache/kafka/pull/2422 KAFKA-4679 Remove unstable markers from Connect APIs @ewencp ignore this PR if you are already started to work on this ticket. You can merge this pull request into a Git repository by running

[GitHub] kafka pull request #2176: KAFKA-4403 Update KafkaBasedLog to use new endOffs...

2016-11-27 Thread baluchicken
GitHub user baluchicken opened a pull request: https://github.com/apache/kafka/pull/2176 KAFKA-4403 Update KafkaBasedLog to use new endOffsets consumer API @ewencp plz review You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] kafka pull request #2161: KAFKA-4307 Inconsistent parameters between console...

2016-11-23 Thread baluchicken
GitHub user baluchicken opened a pull request: https://github.com/apache/kafka/pull/2161 KAFKA-4307 Inconsistent parameters between console producer and consumer @gwenshap please review You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] kafka pull request #2137: KAFKA-1548 Refactor the "replica_id" in requests

2016-11-15 Thread baluchicken
GitHub user baluchicken opened a pull request: https://github.com/apache/kafka/pull/2137 KAFKA-1548 Refactor the "replica_id" in requests You can merge this pull request into a Git repository by running: $ git pull https://github.com/baluchicken/kafka-1

[GitHub] kafka pull request #2083: KAFKA-4318 Migrate ProducerSendTest to the new con...

2016-11-01 Thread baluchicken
GitHub user baluchicken opened a pull request: https://github.com/apache/kafka/pull/2083 KAFKA-4318 Migrate ProducerSendTest to the new consumer You can merge this pull request into a Git repository by running: $ git pull https://github.com/baluchicken/kafka-1 KAFKA-4318