[
https://issues.apache.org/jira/browse/KAFKA-5049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15969442#comment-15969442
]
anugrah edited comment on KAFKA-5049 at 4/14/17 7:42 PM:
---------------------------------------------------------
[~junrao] Hey I am done with the implementation for this. How do I proceed from
here. Should I raise a PR ? Should I post a patch ? What exactly are the steps
to be followed ? Can anybody guide me regarding this ? Thank you for the
pointers as well.
was (Author: anukin):
Hey I am done with the implementation for this. How do I proceed from here.
Should I raise a PR ? Should I post a patch ? What exactly are the steps to be
followed ? Can anybody guide me regarding this ?
> 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)
