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`.


---

Reply via email to