lhotari commented on PR #20176: URL: https://github.com/apache/pulsar/pull/20176#issuecomment-1550784506
@coderzc Please update the PR description with the impact of this bug. Here's my version of describing the impact: MessageRedeliveryController uses the RoaringBitmap under the covers to track entryIds to redeliver. The bug in RoaringBitmap (that has now been fixed) caused at least the case that the bit index 65535 couldn't be set. The consequence of this would be that a message that should be redelivered doesn't get delivered and this could get Key_Shared subscriptions to get stuck. This could impact also other subscription types and cause a backlog to build up. This bug was introduced in 2.10.2 with #17804 since that's when MessageRedeliveryController switched to use the RoaringBitmap based datastructure. @codelipenghui @coderzc Is this the correct interpretation? -- 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-unsubscr...@pulsar.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
