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

[10.1.x] ISPN-13229 LocalPublisherManagerImpl doesn't clean up changeListener … #11560

Merged
merged 3 commits into from Dec 12, 2023
Merged

[10.1.x] ISPN-13229 LocalPublisherManagerImpl doesn't clean up changeListener … #11560

merged 3 commits into from Dec 12, 2023

Conversation

Madhumitha-G-ML
Copy link

Backporting the fix for Memory leak issue in 10.1.x release.

@ryanemerson
Copy link
Contributor

@Madhumitha-G-ML CI is failing checkstyle with the following error:

[2023-12-08T14:00:43.333Z] [ERROR] /home/infinispan/workspace/Infinispan_PR-11560/core/src/main/java/org/infinispan/reactive/publisher/impl/LocalPublisherManagerImpl.java:52:8: Unused import - io.reactivex.internal.functions.Functions. [UnusedImports]

Please remove the import and update your commit.

@ryanemerson ryanemerson changed the title ISPN-13229 LocalPublisherManagerImpl doesn't clean up changeListener … [10.1.x] ISPN-13229 LocalPublisherManagerImpl doesn't clean up changeListener … Dec 8, 2023
@ryanemerson ryanemerson merged commit 7dbc6df into infinispan:10.1.x Dec 12, 2023
1 check failed
@ryanemerson
Copy link
Contributor

Thanks @Madhumitha-G-ML

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ryanemerson ryanemerson ryanemerson left review comments

Assignees
No one assigned
Labels
Changes Required
Projects
None yet
Milestone
No milestone
2 participants
@Madhumitha-G-ML @ryanemerson