Skip to content

Conversation

@LiorKogan
Copy link
Member

Clarify the definition of the trimming threshold for MAXLEN and MINID, specifying that MAXLEN requires a non-negative integer.

Clarify the definition of the trimming threshold for MAXLEN and MINID, specifying that MAXLEN requires a non-negative integer.
@LiorKogan LiorKogan requested a review from dwdougherty December 4, 2025 08:09
@github-actions
Copy link
Contributor

github-actions bot commented Dec 4, 2025

Updated syntax format and string for XTRIM command.
@LiorKogan LiorKogan changed the title Clarify trimming threshold definitions in xtrim.md Clarify trimming threshold definitions in XTRIM and XADD Dec 4, 2025
Clarified the descriptions for MAXLEN and MINID trimming strategies, emphasizing the behavior of Redis during the trimming process.
Clarify the explanation of trimming thresholds for MAXLEN and MINID in xadd.md.
Clarified the description of the 'MAXLEN' threshold in the xtrim command documentation.
Updated formatting for clarity in xtrim command documentation.
@dwdougherty dwdougherty changed the title Clarify trimming threshold definitions in XTRIM and XADD DEV: clarify trimming threshold definitions in XTRIM and XADD Dec 4, 2025
@dwdougherty dwdougherty self-assigned this Dec 4, 2025
Copy link
Collaborator

@dwdougherty dwdougherty left a comment

Choose a reason for hiding this comment

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

Thank you, @LiorKogan! Funny story: the automation we have in place to update data/commands_core.json isn't currently running, so that's why those options were missing in the syntax. Facepalm.

@dwdougherty dwdougherty merged commit fa2712c into main Dec 4, 2025
7 checks passed
@dwdougherty dwdougherty deleted the LiorKogan-patch-1 branch December 4, 2025 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants