[
https://issues.apache.org/jira/browse/KAFKA-5049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15968624#comment-15968624
]
Umesh Chaudhary commented on KAFKA-5049:
----------------------------------------
Thanks [~junrao] for the pointers. While I was looking at the current
implementation, it seems difficult to instantiate ZkPath trivially. Shouldn't
we need another class definition (as a companion of existing ZkPath object)
which enables the instantiation of ZkPath? Please correct me if I am wrong.
> Chroot check should be done for each ZkUtils instance
> -----------------------------------------------------
>
> Key: KAFKA-5049
> URL: https://issues.apache.org/jira/browse/KAFKA-5049
> Project: Kafka
> Issue Type: Bug
> Reporter: Ismael Juma
> Labels: newbie
> Fix For: 0.11.0.0
>
>
> In KAFKA-1994, the check for ZK chroot was moved to ZkPath. However, ZkPath
> is a JVM singleton and we may use multiple ZkClient instances with multiple
> ZooKeeper ensembles in the same JVM (for cluster info, authorizer and
> pluggable code provided by users).
> The right way to do this is to make ZkPath an instance variable in ZkUtils so
> that we do the check once per ZkUtils instance.
> cc [~gwenshap] [~junrao], who reviewed KAFKA-1994, in case I am missing
> something.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
