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:

    @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

View raw message