You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by "lhotari (via GitHub)" <gi...@apache.org> on 2023/02/02 11:43:40 UTC

[GitHub] [pulsar] lhotari commented on pull request #19395: [fix][broker] Release Netty buffer in finally block in ServerCnx#handleLastMessageIdFromCompactedLedger

lhotari commented on PR #19395:
URL: https://github.com/apache/pulsar/pull/19395#issuecomment-1413605606

   > good catch
   > 
   > how did you find the problem ? did you happen to analyse some leak or doing static code analysis manually ?
   
   @eolivelli I was reading and reviewing the code and I happened to spot it. I doubt that this problem happens often enough that it would become an actual problem.


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