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

ByteArrayObjectDataOutput mem leak #115

Closed
wants to merge 2 commits into from
Closed

ByteArrayObjectDataOutput mem leak #115

wants to merge 2 commits into from

Conversation

abeltsios
Copy link

ByteArrayObjectDataOutput were kept per thread (on ThreadLocalOutputCache) and this lead to
memory piling (on clients' side) after heavy usage
(mulitple threads / tasks using Hazelcast client)

for the time being, and as far as I tested it it seems to be working properly.

Larry added 2 commits July 24, 2017 16:51
 memory piling (on client side) after heavy usage
(mulitple threads / tasks using Hazelcast client)

for the time being, and as far as I tested it it seems to be working properly
@asimarslan
Copy link
Contributor

Hi @ampeltsi,

Thanks for finding this bug and submitting the fix. I have created the issue for this problem. For the solution of the leak we don't want to remove the pooling mechanism.

Please refer to this PR for the fix.

@asimarslan asimarslan closed this Oct 13, 2017
@Holmistr Holmistr added the Source: Community Originated from the Community. label Apr 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Source: Community Originated from the Community.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants