lhotari opened a new pull request #10755:
URL: https://github.com/apache/pulsar/pull/10755


   ### Motivation
   
   The repro case in #10738 reproduces some Netty ByteBuf leaks that are 
detected by the Netty Leak detector. There are a few exceptional execution 
paths where the data ByteBuf of OpAddEntry isn't released. 
   
   ### Modifications
   
   - release the data ByteBuf in a try-finally block
   - release the data ByteBuf when execution is rejected in the invalid state 
case where the entry isn't the first in the queue.
   
   This seems to stop the detected leaks in #10738 case, but it doesn't fix the 
issue where a lot of direct memory is consumed and the broker runs of out of 
direct memory.


-- 
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to