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

Fix memory leak when Broker.Open and Broker.Close called repeatedly #1607

Merged
merged 1 commit into from
Feb 19, 2020

Conversation

darklore
Copy link
Contributor

@darklore darklore commented Feb 7, 2020

Broker.registeredMetrics is increased in Broker.Open, but isn't cleared in Broker.Close.
So when Open and Close is called repeatedly it causes memory leak.

@darklore darklore requested a review from bai as a code owner February 7, 2020 00:55
@ghost ghost added the cla-needed label Feb 7, 2020
@darklore
Copy link
Contributor Author

I think I already signed CLA...

@ghost ghost removed the cla-needed label Feb 10, 2020
@dnwe dnwe merged commit 4ee86d9 into IBM:master Feb 19, 2020
@darklore darklore deleted the memory-leak branch February 19, 2020 23:55
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.

2 participants