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 Wed, 05 Sep 2018 13:51:54 GMT
Github user anmolnar commented on the issue:

    https://github.com/apache/zookeeper/pull/611
  
    @enixon I think it leaks the map anyway, because when the new Watcher is added, new Watcher
instance gets created which will be added to Map anyway and the empty one remains forever.
    
    @wangchaod Would you please consider adding a quick unit test for the patch. Otherwise,
it looks good.


---

Mime
View raw message