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.

Sign up for GitHub

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

Jump to bottom

Fix TcpIpConnectionManager_MemoryLeakTest #13959

Merged
merged 1 commit into from
Oct 15, 2018
Merged

Fix TcpIpConnectionManager_MemoryLeakTest #13959

merged 1 commit into from
Oct 15, 2018

Conversation

mmedenjak
Copy link
Contributor

Wait for member to observe membership change before assertion.

Fixes: #13958

Fix TcpIpConnectionManager_MemoryLeakTest
70bb1b8 
Wait for member to observe membership change before assertion.

Fixes: hazelcast#13958
@mmedenjak mmedenjak added this to the 3.11 milestone Oct 15, 2018
@mmedenjak mmedenjak self-assigned this Oct 15, 2018
@taburet taburet self-requested a review October 15, 2018 12:51
@mmedenjak mmedenjak merged commit 71952a6 into hazelcast:master Oct 15, 2018
@mmedenjak
Copy link
Contributor Author

Simple test fix, merging with 1 approval. Thanks for the review @taburet !

@mmedenjak mmedenjak deleted the TcpIpConnectionManager_MemoryLeakTest branch October 15, 2018 13:27
@mmedenjak mmedenjak added the Source: Internal PR or issue was opened by an employee label Apr 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@taburet taburet taburet approved these changes

Assignees

@mmedenjak mmedenjak

Labels
Source: Internal PR or issue was opened by an employee Team: Core Type: Test-Failure
Projects
None yet
Milestone
3.11
Development

Successfully merging this pull request may close these issues.

TcpIpConnectionManager_MemoryLeakTest.test failure
2 participants
@mmedenjak @taburet