zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From lvfangmin <...@git.apache.org>
Subject [GitHub] zookeeper issue #689: ZOOKEEPER-3183:Notifying the WatcherCleaner thread and...
Date Thu, 15 Nov 2018 01:54:13 GMT
Github user lvfangmin commented on the issue:

    https://github.com/apache/zookeeper/pull/689
  
    @anmolnar that's a good point, I think the reason we didn't use ZooKeeperThread or ZooKeeperCriticalThread
is because we don't expect to exit abnormally from the WatcherCleaner thread, which is true
for now. But I think it's better to use ZooKeeperThread or even ZooKeeperCriticalThread to
cover future changes which might cause  the thread exit abnormally.


---

Mime
View raw message