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

Posted to commits@pulsar.apache.org by "TakaHiR07 (via GitHub)" <gi...@apache.org> on 2025/02/14 06:30:51 UTC

Re: [PR] [improve][broker] Optimize high CPU usage when consuming from topics with ongoing txn [pulsar]

TakaHiR07 commented on PR #23189:
URL: https://github.com/apache/pulsar/pull/23189#issuecomment-2658392441

   > This is a quick fix, it is only effective for `managedLedgerNewEntriesCheckDelayInMillis > 0` we can merge pr first and we can continue to review #22944
   
   @coderzc This is not a completely fix. Such as this scene, maxReadPosition < readPosition < LAC, and no new message comming. With this fix, although it would not enter the loop, but then the cursor would be added to waitingCursor. Because no new message comming, the waitingCursor can not be notified, and then the consume process is stuck. Although we then commit the txn to move the maxReadPosition forward, the consume still stuck and remain some message can not be consumed.
   
   Can you help review #22944 ?


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