void-ptr974 commented on PR #26184: URL: https://github.com/apache/pulsar/pull/26184#issuecomment-4981160317
It looks like we currently only cover the `markDeletePosition == lastPosition` case in `testEstimatedUnackedSizeWhenLatestLedgerIsEmpty`. A regression case where the cursor has advanced to `currentLedger:-1` while `lastPosition` is still in the previous ledger would also exercise `isMarkDeletePositionOnEmptyCurrentLedger()` and the invalid-range scenario addressed here. -- 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]
