Errored: apache/geode#5490 (develop - 3cf7caa)

[Travis CI <bu...@travis-ci.org>]

Build Update for apache/geode
-------------------------------------

Build: #5490
Status: Errored

Duration: 7 minutes and 6 seconds
Commit: 3cf7caa (develop)
Author: Bruce Schuchardt
Message: Squashed commit of the following:

commit 70a76592a58379bdd9b53433877b64831fc7432e
Author: Bruce Schuchardt <bs...@pivotal.io>
Date:   Tue Jan 9 08:37:45 2018 -0800

    GEODE-3588 2 restarts of Locator results in split brain

    removed thread dump in new test

commit 89bf34c39f3df4ed7b16d6c9a256e2d26b9d2267
Author: Bruce Schuchardt <bs...@pivotal.io>
Date:   Mon Jan 8 15:57:09 2018 -0800

    GEODE-3588 2 restarts of Locator results in split brain

    Udo's fix for GEODE-870 added a new boolean instance variable to
    GMSJoinLeave to tell its ViewCreator thread to shut down.  This works
    but the state was never being reset after its first use.  This caused
    Subsequent ViewCreator threads to shut down immediately.  The only
    way to fix this condition without a patch is to restart the coordinator node.

    The patch moves this boolean variable to the ViewCreator thread so that
    it is automatically reset when a new ViewCreator is instantiated.

    I also did a little code cleanup, moving GMSJoinLeave methods from the
    end of the file to where its other methods are located and adding
    a setShutdownFlag() method during debugging so I could isolate what
    was happening.

Sarge reviewed the changes for me so this closes #1255

View the changeset: https://github.com/apache/geode/compare/ff008e33a6c3...3cf7caab3c37

View the full build log and details: https://travis-ci.org/apache/geode/builds/326899091?utm_source=email&utm_medium=notification

--

You can configure recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications