-
Notifications
You must be signed in to change notification settings - Fork 316
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
Etcd memory leak when using jetcd-core 0.7.6 #1240
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have test it and it can fix #1236
608fbe7
to
59e0ab0
Compare
int after = extractGoRoutinesCount(uri); | ||
assertThat(after).isGreaterThan(0); | ||
|
||
assertThat(after).isGreaterThan(before); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if this makes this a flake test as there will be some fluctuations here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm looking into that
1978abd
to
df3aa33
Compare
ee31fc6
to
61c53fb
Compare
Signed-off-by: Luca Burgazzoli <lburgazzoli@gmail.com>
Fixes #1236