You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@helix.apache.org by narendly <gi...@git.apache.org> on 2018/07/16 22:03:28 UTC

[GitHub] helix pull request #247: Fix TestAssignableInstanceManagerControllerSwitch

GitHub user narendly opened a pull request:

    https://github.com/apache/helix/pull/247

    Fix TestAssignableInstanceManagerControllerSwitch

    This test uses RoutingTableProvider. It was not being shut down, causing an ExecutorService thread to continue executing periodic updates. This RB makes it so that it is shut down at the end of the test.
    
    Changelist:
    1. Add shutdown() on RoutingTableProvider at the end of the test

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

    $ git pull https://github.com/narendly/helix 1346194

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

    https://github.com/apache/helix/pull/247.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 #247
    
----
commit bd171f26d164dc33ac91fc393d11de269c2665b0
Author: Hunter Lee <na...@...>
Date:   2018-07-16T22:02:47Z

    [HELIX-733] Fix TestAssignableInstanceManagerControllerSwitch
    
    This test uses RoutingTableProvider. It was not being shut down, causing an ExecutorService thread to continue executing periodic updates. This RB makes it so that it is shut down at the end of the test.
    
    Changelist:
    1. Add shutdown() on RoutingTableProvider at the end of the test

----


---

[GitHub] helix pull request #247: Fix TestAssignableInstanceManagerControllerSwitch

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/helix/pull/247


---