liangyepianzhou commented on PR #24222: URL: https://github.com/apache/pulsar/pull/24222#issuecomment-2933231905
> Out of the scope of this PR, I think it would be better to add a method to `ManagedLedger` for it. Because the customized ML implementation (e.g. StreamNative's Ursa engine) might not depend on the broker entry metadata to get the message's index. It could be another PIP. Yes, we should add a new manager ledger interface to support customized ML implementations. Additionally, the current message index retrieval solution has suboptimal performance. We could improve this by adding a field in `LedgerInfo` to record the first entry's index, then compute subsequent indexes directly from message IDs. I've created an [issue](https://github.com/apache/pulsar/issues/24374) to track this enhancement. -- 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]
