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: [email protected]

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

Reply via email to