kaisun2000 commented on a change in pull request #1035:
URL: https://github.com/apache/helix/pull/1035#discussion_r431594622
##########
File path:
zookeeper-api/src/main/java/org/apache/helix/zookeeper/zkclient/ZkClient.java
##########
@@ -1074,6 +1101,34 @@ private Stat getStat(final String path, final boolean
watch) {
}
}
+ /*
+ * Install watch if there is such node and return the stat
+ * Don't install watch if there is no such node and return null
+ *
+ * Use ZooKeeper native api getData() as underlying method
+ */
+ private Stat getStat2(final String path) {
Review comment:
getStatNoInstallingWatchWhenNodeNotExists()? this seems to be too long.
Or maybe getStatNoWatchNoNode()?
Any better name?
----------------------------------------------------------------
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
