道君- Tao Jiuming
道君- Tao Jiuming
@lhotari I updated the description
> Can I help merge this PR? This PR needs more review
@lhotari Rebased, please help review when you have time
close reopen to trigger the CI checks
> @dao-jun I pushed [914b2ca](https://github.com/apache/pulsar/commit/914b2ca4bc4c2f3ee190cbf696458d796fddb705) to this PR to avoid breaking the ManagedCursor interface. This way we can make this change to Pulsar 4.0 without needing a PIP. Thanks
> @dao-jun To speed up the review, I pushed the changes for adding a configuration parameter `managedLedgerCursorResetLedgerCloseTimestampMaxClockSkewMillis` and logic to handle it. That was something that I was concerned about....
> @dao-jun There's a bug report that might be related to this change. Do you have a chance to check #23910? Thanks Will do it after a few days~
LGTM, but maybe the change needs a PIP?
firstly, I dont think it's a bug. This is an incorrect usage of the messaging system. A message should be acked after consumed, not before. and then, if we fix...
it seems this is expected behavior