[GitHub] [pulsar] eolivelli opened a new pull request, #17995: [bug][broker] fix memory leak in case of error conditions in PendingReadsManager

[GitBox <gi...@apache.org>]

eolivelli opened a new pull request, #17995:
URL: https://github.com/apache/pulsar/pull/17995

   ### Motivation
   
   In case of error conditions we are not releasing the entries returned by partial reads. 
   
   ### Modifications
   
   Release the entries on the error handling paths.
   
   ### Verifying this change
   
   This change is a trivial rework / code cleanup without any test coverage.
   
   ### Documentation
   
   <!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. -->
   
   - [ ] `doc` <!-- Your PR contains doc changes -->
   - [ ] `doc-required` <!-- Your PR changes impact docs and you will update later -->
   - [x] `doc-not-needed` <!-- Your PR changes do not impact docs -->
   - [ ] `doc-complete` <!-- Docs have been already added -->
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] eolivelli merged pull request #17995: [fix][broker] Fix memory leak in case of error conditions in PendingReadsManager

[GitBox <gi...@apache.org>]

eolivelli merged PR #17995:
URL: https://github.com/apache/pulsar/pull/17995


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org