rondagostino commented on pull request #10550:
URL: https://github.com/apache/kafka/pull/10550#issuecomment-827032633


   > It might be helpful to move the znode setup code into KafkaZkClient
   
   @cmccabe One possibility is that the first time we connect to ZooKeeper we 
connect without the chroot path, create that path if it doesn't exist, and then 
disconnect -- then (and thereafter)we connect with the chroot path?  If we did 
that then I think it would have to happen in `kafka.zookeeper.ZooKeeperClient` 
rather than in `KafkaZkClient` since the former is where we provide the chroot 
path -- the latter seems to simply use the former with no concept of a chroot 
path.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to