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

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/06/19 03:25:31 UTC

[GitHub] [pulsar] congbobo184 edited a comment on pull request #10966: [Broker] Fix get wrong backlog number.

congbobo184 edited a comment on pull request #10966:
URL: https://github.com/apache/pulsar/pull/10966#issuecomment-864348105


   > I looked at the method `getNumberOfEntries` of the class `ManagedLedgerImpl`, if the `fromPosition` or `toPosition` is invalid, such as `3:-1`, the compute result will be wrong. If use the `openClosed` range will work around the problem. Maybe we could refine this method in next PR.
   
   yes, now after use `getNumberOfEntries`  will `-1` so we should find all the place where use `getNumberOfEntries` change it don't `-1` in the next pr


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org