You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zookeeper.apache.org by anmolnar <gi...@git.apache.org> on 2018/10/09 04:52:29 UTC
[GitHub] zookeeper pull request #660: ZOOKEEPER-2320. C-client crashes when removing ...
GitHub user anmolnar opened a pull request:
https://github.com/apache/zookeeper/pull/660
ZOOKEEPER-2320. C-client crashes when removing watcher asynchronously in "local" mode
Trying to fix invalid point issue happening intermittently in the PreCommit job. Looks like this old patch fixes the issue in the C client, so let's try to re-animate it.
I'm not a C expert, so waiting for at least 2 committers to approve.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/anmolnar/zookeeper ZOOKEEPER-2320
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zookeeper/pull/660.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #660
----
commit daf35ce2c355fc5cdc73e0f7baf0cba5bbaeab8f
Author: Andor Molnar <an...@...>
Date: 2018-10-08T14:54:29Z
ZOOKEEPER-2320. Fix invalid pointer issue on zoo_aremove_watchers()
----
---
[GitHub] zookeeper issue #660: ZOOKEEPER-2320. C-client crashes when removing watcher...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit commented on the issue:
https://github.com/apache/zookeeper/pull/660
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2386/
---
[GitHub] zookeeper pull request #660: ZOOKEEPER-2320. C-client crashes when removing ...
Posted by anmolnar <gi...@git.apache.org>.
Github user anmolnar closed the pull request at:
https://github.com/apache/zookeeper/pull/660
---
[GitHub] zookeeper issue #660: ZOOKEEPER-2320. C-client crashes when removing watcher...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit commented on the issue:
https://github.com/apache/zookeeper/pull/660
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2387/
---
[GitHub] zookeeper issue #660: ZOOKEEPER-2320. C-client crashes when removing watcher...
Posted by anmolnar <gi...@git.apache.org>.
Github user anmolnar commented on the issue:
https://github.com/apache/zookeeper/pull/660
@lvfangmin Assertion failied in the unit test `testRemoveWatchers`. Investigating.
---
[GitHub] zookeeper issue #660: ZOOKEEPER-2320. C-client crashes when removing watcher...
Posted by anmolnar <gi...@git.apache.org>.
Github user anmolnar commented on the issue:
https://github.com/apache/zookeeper/pull/660
retest this please
---
[GitHub] zookeeper issue #660: ZOOKEEPER-2320. C-client crashes when removing watcher...
Posted by lvfangmin <gi...@git.apache.org>.
Github user lvfangmin commented on the issue:
https://github.com/apache/zookeeper/pull/660
@anmolnar do you know why it failing the check here? From the latest Jenkins build link I cannot find any failure.
---