Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IGNITE-6179: fixed tests with client reconnect #2596

Closed
wants to merge 1 commit into from

Conversation

vkuragin
Copy link
Contributor

@vkuragin vkuragin commented Sep 5, 2017

Affects:
DynamicIndexPartitionedAtomicConcurrentSelfTest.testClientReconnectWithCacheRestart()
DynamicIndexPartitionedTransactionalConcurrentSelfTest.testClientReconnectWithCacheRestart()
DynamicIndexReplicatedTransactionalConcurrentSelfTest.testClientReconnectWithCacheRestart()
DynamicIndexReplicatedAtomicConcurrentSelfTest.testClientReconnectWithCacheRestart()

devozerov pushed a commit to gridgain/apache-ignite that referenced this pull request Sep 6, 2017
…lfTest.testClientReconnectWithCacheRestart. This closes apache#2596.
@asfgit asfgit closed this in 1202f3f Sep 6, 2017
asfedotov pushed a commit to gridgain/apache-ignite that referenced this pull request Oct 13, 2017
…lfTest.testClientReconnectWithCacheRestart. This closes apache#2596.
vkuragin added a commit to vkuragin/ignite that referenced this pull request Apr 23, 2018
…lfTest.testClientReconnectWithCacheRestart. This closes apache#2596.
ygerzhedovich pushed a commit to gridgain/apache-ignite that referenced this pull request Mar 13, 2023
…vers sometimes hangs forever (apache#2596)

The problem is that, if a grid fails to start (for instance, due to PME), then the test times out, then afterTest() is called which calls stopAllGrids(), which, in turn, waits for all grids to be started, so it hangs as well, this time forever (as the timeout is not applied to afterTest()).

We try to solve the problem by using the variation of stopAllGrids() that does not wait for the grids to start and tries to stop them regargless of their state.

Cherry-picked from commit 97cf70c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant