lhotari commented on code in PR #973: URL: https://github.com/apache/pulsar-site/pull/973#discussion_r1830872585
########## docs/cookbooks-retention-expiry.md: ########## @@ -205,7 +205,7 @@ The following retention policies are available: | `consumer_backlog_eviction` | The broker will begin discarding backlog messages | > #### Beware the distinction between retention policy types -> As you may have noticed, there are two definitions of the term "retention policy" in Pulsar, one that applies to persistent storage of messages not in backlogs, and one that applies to messages within backlogs. +> As you may have noticed, the scope of retention includes the backlog, so retention must be greater than the backlog quota; otherwise, Pulsar will throw an error when set both. Review Comment: Fix grammar in this sentence. "when set both" is not correct. "As you may have noticed" could be omitted. Suggestion: ``` :::note The scope of retention includes the backlog, so retention must be greater than the backlog quota. Otherwise, Pulsar will throw an error when both are set. ::: ``` -- 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]
