[GitHub] kafka pull request: MINOR: MemoryRecords.sizeInBytes throws NPE wh...

2016-01-18 Thread dajac
GitHub user dajac opened a pull request: https://github.com/apache/kafka/pull/786 MINOR: MemoryRecords.sizeInBytes throws NPE when non-writable. I just noticed that `MemoryRecords.sizeInBytes` throws NPE when MemoryRecords is non-writable. `compressor` is explicitly set to null

[GitHub] kafka pull request: KAFKA-3098: "partition.assignment.strategy" ap...

2016-01-14 Thread dajac
GitHub user dajac opened a pull request: https://github.com/apache/kafka/pull/774 KAFKA-3098: "partition.assignment.strategy" appears twice in documentation You can merge this pull request into a Git repository by running: $ git pull https://github.com/dajac/kafka

[GitHub] kafka pull request: KAFKA-3085: BrokerChangeListener computes inco...

2016-01-11 Thread dajac
GitHub user dajac opened a pull request: https://github.com/apache/kafka/pull/752 KAFKA-3085: BrokerChangeListener computes inconsistent live/dead broker list You can merge this pull request into a Git repository by running: $ git pull https://github.com/dajac/kafka KAFKA

[GitHub] kafka pull request: KAFKA-3085: BrokerChangeListener computes inco...

2016-01-11 Thread dajac
GitHub user dajac opened a pull request: https://github.com/apache/kafka/pull/756 KAFKA-3085: BrokerChangeListener computes inconsistent live/dead broker list. Follow up PR as per comments in the ticket. @junrao It should be correct now as `curBrokers` included only live

[GitHub] kafka pull request: MINOR: Tiny optimization to avoid mapping twic...

2015-12-22 Thread dajac
GitHub user dajac opened a pull request: https://github.com/apache/kafka/pull/714 MINOR: Tiny optimization to avoid mapping twice on the results. As pointed out by @granthenke in #196, error can be wrapped in JShort directly to avoid second map. You can merge this pull request

[GitHub] kafka pull request: KAFKA-2072: Replace StopReplica Request/Respon...

2015-09-05 Thread dajac
GitHub user dajac opened a pull request: https://github.com/apache/kafka/pull/196 KAFKA-2072: Replace StopReplica Request/Response with their org.apache.kafka.common.requests equivalents You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] kafka pull request: KAFKA-2072: Add StopReplica request/response t...

2015-08-26 Thread dajac
GitHub user dajac opened a pull request: https://github.com/apache/kafka/pull/170 KAFKA-2072: Add StopReplica request/response to o.a.k.common.requests This PR adds StopReplica request and response as it is required by @ijuma for KAFKA-2411. Migration of core module is addressed

[GitHub] kafka pull request: KAFKA-2072 [WIP]: Add StopReplica request/resp...

2015-08-24 Thread dajac
Github user dajac closed the pull request at: https://github.com/apache/kafka/pull/141 --- 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: KAFKA-2072 [WIP]: Add StopReplica request/resp...

2015-08-15 Thread dajac
GitHub user dajac opened a pull request: https://github.com/apache/kafka/pull/141 KAFKA-2072 [WIP]: Add StopReplica request/response to o.a.k.common.requests and replace the usage in core module Migration is done but this PR will need to be rebased on #110. I have copied some

[GitHub] kafka pull request: KAFKA-2071: Replace Producer Request/Response ...

2015-08-04 Thread dajac
GitHub user dajac opened a pull request: https://github.com/apache/kafka/pull/110 KAFKA-2071: Replace Producer Request/Response with their org.apache.kafka.common.requests equivalents This PR replaces all producer request/response with their common equivalents but doesn't touch