315157973 opened a new pull request, #21939: URL: https://github.com/apache/pulsar/pull/21939
Main Issue: #xyz ### Motivation When the client machine's clock is incorrect (eg: set to 1 year later) and the Broker does not set the AppendBrokerTimestampMetadataInterceptor, the Ledger will not be cleaned up. Because if the Broker's timestamp is not set, the expiration check will be based on the client's publish time. As we discuss in https://lists.apache.org/thread/0vtjt436xvlg0fr4ogchkjdvjcp56mvz ### Modifications Enable `AppendBrokerTimestampMetadataInterceptor` by default ### Verifying this change - [x] Make sure that the change passes the CI checks. ### Does this pull request potentially affect one of the following parts: - [x] The default values of configurations ### Documentation - [x] `doc-required` <!-- Your PR changes impact docs and you will update later --> ### Matching PR in forked repository PR in forked repository: <!-- ENTER URL 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]
