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 #689: ZOOKEEPER-3183:Notifying the WatcherCleaner thread and...
Date Wed, 14 Nov 2018 20:31:24 GMT
Github user anmolnar commented on the issue:

    https://github.com/apache/zookeeper/pull/689
  
    @tumativ I tend to agree with @lvfangmin in adding `interrupt()` call to shutdown would
be the easiest solution here. 
    
    In which case I would also add `break;` to the InterruptedException handler of addDeadWatcher()
method.
    
    Out of interest, why WatcherCleaner is not inherited from `ZooKeeperThread` btw?


---

Mime
View raw message