-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Bug] [ml] Incorrect behavior of Topic Retention Policy #22473
Comments
@lhotari PTAL |
Good description @dao-jun . This makes sense. We need to ask the original developer of the solution for clarification whether this is a bug or not. I guess |
@lhotari According to https://pulsar.apache.org/docs/3.2.x/cookbooks-retention-expiry/#retention-policies and ManagedLedgerImpl#internalTrimLedgers, delete a ledger or not should be based on |
+1 @dao-jun the reported issue seems to be a bug based on this information. The definition of the retention settings was explicitly mentioned.
|
In the first version doc, it says: messages in all topics in the namespace, even acknowledged messages, will be retained. |
This is a doc issue, since https://github.com/apache/pulsar/pull/5482/files |
Search before asking
Read release policy
Version
master branch
Minimal reproduce step
What did you expect to see?
Ledgers [1, 3] deleted.
What did you see instead?
Ledgers [1, 5] deleted
Anything else?
No response
Are you willing to submit a PR?
The text was updated successfully, but these errors were encountered: