void-ptr974 commented on PR #25984: URL: https://github.com/apache/pulsar/pull/25984#issuecomment-4697487022
I think there is still an issue here: `getScheduledMessages()` only skips entries before `firstLiveLedgerId` when `!containsMessage(...)`. For old-ledger entries that are still present in the bucket bitmap, `containsMessage(...)` returns true, so those deleted-ledger positions can still be returned. The key point is that `ledgerId < firstLiveLedgerId` should be enough to skip the entry, and the tracker state/counter should stay consistent. -- 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]
