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(BroadcastProcessor): prevent memory leaks by clearing active subscriptions #1536

Merged
merged 1 commit into from
Mar 6, 2024

Conversation

jponge
Copy link
Member

@jponge jponge commented Mar 4, 2024

Refs: #1532

@jponge jponge added backport PR that shall be back-ported bug Something isn't working labels Mar 4, 2024
@jponge jponge requested a review from cescoffier March 4, 2024 18:31
Copy link

codecov bot commented Mar 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.44%. Comparing base (3c6b3f4) to head (6a2c6f0).
Report is 35 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##               main    #1536   +/-   ##
=========================================
  Coverage     89.43%   89.44%           
+ Complexity     3282     3271   -11     
=========================================
  Files           457      457           
  Lines         13264    13274   +10     
  Branches       1637     1644    +7     
=========================================
+ Hits          11863    11873   +10     
  Misses          788      788           
  Partials        613      613           
Files Coverage Δ
...operators/multi/processors/BroadcastProcessor.java 90.47% <100.00%> (+0.47%) ⬆️

... and 9 files with indirect coverage changes

@jponge jponge marked this pull request as ready for review March 5, 2024 10:39
@jponge jponge merged commit 52050e8 into smallrye:main Mar 6, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport PR that shall be back-ported bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants