You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@zookeeper.apache.org by lvfangmin <gi...@git.apache.org> on 2018/07/21 18:41:21 UTC

[GitHub] zookeeper issue #504: Potential NPE in Observer#observeLeader and Follower#f...

Github user lvfangmin commented on the issue:

    https://github.com/apache/zookeeper/pull/504
  
    The newly added exception will be caught in the QuorumPeer, which will trigger another round of look up.
    
    This change looks good to me, it’s always safer to check, although this may not actually happen on 3.6 with dynamic config, unless there is a bug, which we should fix.
    
    +1


---