[ 
https://issues.apache.org/jira/browse/KAFKA-5049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15969805#comment-15969805
 ] 

Umesh Chaudhary commented on KAFKA-5049:
----------------------------------------

[~anukin] you can send a PR and for detailed steps you can refer [this 
page|https://cwiki.apache.org/confluence/display/KAFKA/Contributing+Code+Changes#ContributingCodeChanges-PullRequest]
 

> 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)

Reply via email to