curator-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Randgalt <...@git.apache.org>
Subject [GitHub] curator pull request: [CURATOR-299] WatcherRemovalManager must be ...
Date Thu, 21 Apr 2016 01:15:54 GMT
GitHub user Randgalt opened a pull request:

    https://github.com/apache/curator/pull/145

    [CURATOR-299] WatcherRemovalManager must be updated when there's an exception, retry,
etc.

    If there's an exeption, retries exceeded, etc., the watcher was still being stored in
WatcherRemovalManager thus causing it to be incorrectly removed. This fix removes the watch
from the WatcherRemovalmanager on exceptions, retries exceeded, etc.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/apache/curator CURATOR-299

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/curator/pull/145.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 #145
    
----
commit b1d2198a75d6e01b48789635ba648bedc542c439
Author: randgalt <randgalt@apache.org>
Date:   2016-04-21T01:12:31Z

    If there's an exeption, retries exceeded, etc., the watcher was still being stored in
WatcherRemovealManager thus causing it to be incorrectly removed. This fix removes the watch
from the WatcherRemovalmanager on exceptions, retries exceeded, etc.

commit e76eb590fddb5b363b3d40b76908d40f2a89ffc7
Author: randgalt <randgalt@apache.org>
Date:   2016-04-21T01:13:42Z

    Merge branch 'CURATOR-3.0' into CURATOR-299

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message