zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ZOOKEEPER-2978) missing list
Date Thu, 15 Feb 2018 22:07:00 GMT

    [ https://issues.apache.org/jira/browse/ZOOKEEPER-2978?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16366324#comment-16366324
] 

ASF GitHub Bot commented on ZOOKEEPER-2978:
-------------------------------------------

Github user afine commented on the issue:

    https://github.com/apache/zookeeper/pull/461
  
    @achimbab Thanks for the patch.
    
    I'm wondering if you ran into this bug while running ZooKeeper? This code is pretty old
and should never actually be executed (for reasons described here: https://github.com/apache/zookeeper/pull/461/files#diff-e08c1cfd3802a0c3156847175ab8e24bR514).

    
    I would be happy to merge in this fix but please rename the JIRA and the pull request
to something more descriptive (as I do not have permissions to change the name of a Pull Request).
Something like "fix potential null pointer exception when deleting node".


> missing list
> ------------
>
>                 Key: ZOOKEEPER-2978
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2978
>             Project: ZooKeeper
>          Issue Type: Bug
>          Components: java client
>    Affects Versions: 3.4.11
>            Reporter: achimbab
>            Priority: Trivial
>             Fix For: 3.4.12
>
>
> At line 518, 'existWatches.remove(clientPath)' is null because watches for clientPath
is already removed.
> https://github.com/apache/zookeeper/pull/461/commits/a6044af23ae1096a8c5305633320fa139cf730b2
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Mime
View raw message