rocketraman commented on a change in pull request #5482: [Issue 5309][docs] Clarify wording around retention policies URL: https://github.com/apache/pulsar/pull/5482#discussion_r339668922
########## File path: site2/docs/cookbooks-retention-expiry.md ########## @@ -31,7 +31,7 @@ By default, when a Pulsar message arrives at a broker it will be stored until it Retention policies are particularly useful if you intend to exclusively use the Reader interface. Because the Reader interface does not use acknowledgements, messages will never exist within backlogs. Most realistic Reader-only use cases require that retention be configured. -When you set a size limit of, say, 10 gigabytes, then messages in all topics in the namespace, *even acknowledged messages*, will be retained until the size limit for the topic is reached; if you set a time limit of, say, 1 day, then messages for all topics in the namespace will be retained for 24 hours. +When you set a size limit of, say, 10 gigabytes, then acknowledged messages in all topics in the namespace will be retained until the size limit for the topic is reached; if you set a time limit of, say, 1 day, then acknowledged messages for all topics in the namespace will be retained for 24 hours. The retention settings do not affect unacknowledged messages -- these are instead controlled by the backlog quota and retention policy. Review comment: If I'm reading that correctly, that means there are 3 cases: 1) Messages not in a subscription backlog, and not acked: retention policy applies. 2) Messages in a subscription backlog, and not acked: backlog policy applies. 3) Messages acked by one or more subscriptions: retention policy applies. Is that right? If so, then I think it might be worth just adding those three cases into the docs. ---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
