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 code (ef669a5) for now.
I'd appreciate feedback on it mainly around how I handle things in the
ControllerChannelManager. I have introduced a new 'sendRequest' method for
o.a.k.common.requests and kept the old one for compatibility reason. We'll be
able to remove the old one in the future when migration of all requests and
responses to o.a.k.common.requests is completed.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/dajac/kafka KAFKA-2072
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/kafka/pull/141.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #141
----
commit 22f2466f985cde1787a41f13a0191a538fc3a23f
Author: David Jacot <david.ja...@gmail.com>
Date: 2015-08-13T15:36:49Z
Add o.a.k.c.r.StopReplicaRequest and o.a.k.c.r.StopReplicaResponse.
commit 3a31ba9dc3d912e91535cf3a4e373b8f56b347b4
Author: David Jacot <david.ja...@gmail.com>
Date: 2015-08-13T17:15:18Z
Replace k.a.StopReplicaRequest and k.a.StopReplicaResponse in KafkaApis by
their org.apache.kafka.common.requests equivalents.
commit ef669a5ff5fc125624d5d1ec79b92940d43ca3bb
Author: David Jacot <david.ja...@gmail.com>
Date: 2015-08-14T18:42:37Z
Code cherry-picked from KAFKA-2071. It can be removed when KAFKA-2071 is
merged.
commit cbaa987385d989fad8cc3f40d50a24c2ee25ae78
Author: David Jacot <david.ja...@gmail.com>
Date: 2015-08-14T18:46:21Z
Replace k.a.StopReplicaRequest and k.a.StopReplicaResponse in Controller by
their org.apache.kafka.common.requests equivalents.
commit 48a05d81c94ca30fff96df8c82587e64db4260b0
Author: David Jacot <david.ja...@gmail.com>
Date: 2015-08-14T18:53:32Z
Remove k.a.StopReplicaRequest and k.a.StopReplicaResponse.
----
---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
