murong00 commented on pull request #1359: URL: https://github.com/apache/incubator-kyuubi/pull/1359#issuecomment-964918621
> Can we make this change directly to ServiceDiscovery.createServiceNode? For example, when `makePath` for the namespace `val ns = ZKPaths.makePath(null, namespace)`, we can mix the chroot part in? @yaooqinn @ulysses-you I have tried this, however the `_zkClient` is inited by `localhost:2181/chroot`, if we use `val ns = ZKPaths.makePath(chroot, namespace)` it will lead to exception `NoNode for /chroot/kyuubi`, use a `_zkClient` inited by `localhost:2181` is correct to create 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
