Skip to content

Fix leak when cleaning transactions up #1133

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

Closed
wants to merge 1 commit into from

Conversation

elrodro83
Copy link

With the commit d500dd7 that was release on 1.9.34, there are cases in which the cleanup is not properly done or not done at all, causing a degradation in performance over time.
This PR addresses that issue.

@oleksiys
Copy link
Contributor

oleksiys commented Apr 6, 2016

Good point!
I've decided to rework the previous fix, will ping you once it's commited.

Thanks a lot!

oleksiys pushed a commit that referenced this pull request Apr 7, 2016
@oleksiys
Copy link
Contributor

oleksiys commented Apr 7, 2016

please let me know if the fix works for you.
thank you!

@elrodro83
Copy link
Author

That commit fixed the leak, thanks!

@elrodro83 elrodro83 closed this Apr 7, 2016
@elrodro83 elrodro83 deleted the transaction-leak branch April 8, 2016 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants