zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From anmolnar <...@git.apache.org>
Subject [GitHub] zookeeper issue #611: ZOOKEEPER-3131
Date Fri, 21 Sep 2018 08:46:05 GMT
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) 


---

Mime
View raw message