Github user hanm commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/590#discussion_r219644285
--- Diff: src/java/main/org/apache/zookeeper/server/watch/WatchManager.java
---
@@ -180,7 +192,8 @@ synchronized void dumpWatches(PrintWriter pwriter,
boolean byPath) {
* watcher object reference
* @return true if the watcher exists, false otherwise
*/
--- End diff --
we should remove this comment, which is already present in the
`IWatchManager`, also for consistency with other overrides in `WatchManager`.
---