Github user anmolnar commented on the issue:

    https://github.com/apache/zookeeper/pull/611
  
    @lvfangmin Let me try to wrap what has been discussed with this issue so 
far.
    - You're saying that #611 is not needed, because this PR fixes it and don't 
need to maintain `watch2Paths` explicitly,
    - If that's true and this patch targets Netty only, which patch has fixed 
the NIO side?
    - One of our customers reported inconsistent watch reporting in 4lw 
commands which I successfully fixed with #611 . Are sure it's still not needed 
given that it should be only a reporting issue?
    (`wchc` command uses `watch2paths` for reporting, so if it still contains 
watchers which are not effective anymore, the report is meaningless and might 
be misleading) 


---

Reply via email to