kezhuw opened a new pull request, #1996: URL: https://github.com/apache/zookeeper/pull/1996
Previously, get config watcher behaved different in chroots: * In chroot "/short", the watcher receives "/zookeeper/config" after ZOOKEEPER-4565. Before that, it got abnormal path "eper/config". * In chroot "/pretty-long-chroot-path", the watcher receives "/zookeeper/config". * In chroot "/zoo", the watcher receives abnormal path "keeper/config". ZOOKEEPER-4565 should fix this, but failed to do so. * In chroot "/zookeeper" or "/zookeeper/config", the watcher got no event as `getConfig` register watcher in path "/zookeeper/config". But config node path got stripped to "/config" or "/" before event delivery. This commit fixes get config watcher path to "/zookeeper/config". In long term, it might be better to move chroot out of ClientCnxn as ZOOKEEPER-838 suggested, but it is apparently a much bigger change. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
