[
https://issues.apache.org/jira/browse/KAFKA-3210?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15206771#comment-15206771
]
Grant Henke commented on KAFKA-3210:
------------------------------------
[~fpj] I would be curious on your thoughts on other client libraries that
package session abstraction and automated failure handling and why implementing
via the raw api is the best choice. Would utilizing a project like Apache
Curator make sense here and result in less zookeeper code in Kafka directly? I
am not challenging the proposal but asking to understand and be more informed.
> Using asynchronous calls through the raw ZK API in ZkUtils
> ----------------------------------------------------------
>
> Key: KAFKA-3210
> URL: https://issues.apache.org/jira/browse/KAFKA-3210
> Project: Kafka
> Issue Type: Improvement
> Components: controller, zkclient
> Affects Versions: 0.9.0.0
> Reporter: Flavio Junqueira
>
> We have observed a number of issues with the controller interaction with
> ZooKeeper mainly because ZkClient creates new sessions transparently under
> the hood. Creating sessions transparently enables, for example, old
> controller to successfully update znodes in ZooKeeper even when they aren't
> the controller any longer (e.g., KAFKA-3083). To fix this, we need to bypass
> the ZkClient lib like we did with ZKWatchedEphemeral.
> In addition to fixing such races with the controller, it would improve
> performance significantly if we used the async API (see KAFKA-3038). The
> async API is more efficient because it pipelines the requests to ZooKeeper,
> and the number of requests upon controller recovery can be large.
> This jira proposes to make these two changes to the calls in ZkUtils and to
> do it, one path is to first replace the calls in ZkUtils with raw async ZK
> calls and block so that we don't have to change the controller code in this
> phase. Once this step is accomplished and it is stable, we make changes to
> the controller to handle the asynchronous calls to ZooKeeper.
> Note that in the first step, we will need to introduce some new logic for
> session management, which is currently handled entirely by ZkClient. We will
> also need to implement the subscription mechanism for event notifications
> (see ZooKeeperLeaderElector as a an exemple).
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
