GitHub user borjab reopened a pull request:

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

    [Fix] Curator-444

    This ensures that not two nodes are simultaneusly as leaders. 
    
    If setLeadership(false) is redundant no event would be reported anyway so 
it is quite safe.

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

    $ git pull https://github.com/borjab/curator CURATOR-444

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

    https://github.com/apache/curator/pull/243.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 #243
    
----
commit 9b18a0bb67a48efec6cecfa530b41b1d5799bd5a
Author: Borja Bravo <[email protected]>
Date:   2017-11-28T10:21:32Z

    [Fix] Curator-444

commit b23c302592b035a8105c9ac71633608c1178765f
Author: Borja Bravo <[email protected]>
Date:   2017-11-28T10:42:07Z

    [Test] Test skeleton

commit d4495a43f0d9114306e943d669ae57810fff9cd6
Author: Borja Bravo <[email protected]>
Date:   2017-11-28T15:57:58Z

    [Test] working test. Still does not reproduce error.

commit a95b10fb9677d45d3dfeba32bca52f50545aaf78
Author: Borja Bravo <[email protected]>
Date:   2017-11-28T16:25:50Z

    [Test] Test reproduces error (without the previous fix)

commit d0258cf54b88dc91bf42e5208e3acfc6c4e23e88
Author: Borja Bravo <[email protected]>
Date:   2017-11-28T17:31:40Z

    [Test] Improved test. Reset after close

----


---

Reply via email to