Skip to content

fix: change minimum retention values to -1#616

Open
jguddas wants to merge 2 commits into
asyncapi:masterfrom
jguddas:patch-1
Open

fix: change minimum retention values to -1#616
jguddas wants to merge 2 commits into
asyncapi:masterfrom
jguddas:patch-1

Conversation

@jguddas
Copy link
Copy Markdown

@jguddas jguddas commented Feb 18, 2026

Updated minimum values for time and size properties to allow for infinite retention.

Time limit Size limit Message retention
-1 -1 Infinite retention
-1 >0 Based on the size limit
>0 -1 Based on the time limit
0 0 Disable message retention, which means messages are not reserved (by default)
0 >0 Invalid
>0 0 Invalid
>0 >0 Acknowledged messages or messages with no active subscription will not be retained when either time or size reaches the limit.

https://pulsar.apache.org/docs/next/cookbooks-retention-expiry

Updated minimum values for time and size properties to allow for infinite retention.
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Welcome to AsyncAPI. Thanks a lot for creating your first pull request. Please check out our contributors guide useful for opening a pull request.
Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.

Removed specific retention details from time and size descriptions.
@sonarqubecloud
Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant