Skip to content

Minor edits#3559

Open
LiorKogan wants to merge 5 commits into
mainfrom
LiorKogan-patch-1
Open

Minor edits#3559
LiorKogan wants to merge 5 commits into
mainfrom
LiorKogan-patch-1

Conversation

@LiorKogan

@LiorKogan LiorKogan commented Jun 28, 2026

Copy link
Copy Markdown
Member

Clarify usage of the XSETID command in documentation.


Note

Low Risk
Markdown documentation only; no runtime or API behavior changes.

Overview
Documentation-only updates to three stream command pages.

XSETID and XIDMPRECORD intros now follow the same pattern: state they are internal, warn users not to call them directly, then briefly explain what Redis uses them for (XSETID on primary shards for replicating the stream’s last delivered entry ID; XIDMPRECORD for IDMP metadata replayed during AOF load).

XCFGSET adds a link to the idempotent message processing developer guide and tightens IDMP-DURATION wording (“retained” / “in seconds”).

Reviewed by Cursor Bugbot for commit ee774a4. Bugbot is set up for automated code reviews on this repo. Configure here.

Clarify usage of the XSETID command in documentation.
@github-actions

github-actions Bot commented Jun 28, 2026

Copy link
Copy Markdown
Contributor

@jit-ci

jit-ci Bot commented Jun 28, 2026

Copy link
Copy Markdown

🛡️ Jit Security Scan Results

CRITICAL HIGH MEDIUM

✅ No security findings were detected in this PR


Security scan by Jit

Clarified the description of the XIDMPRECORD command and its usage.
@LiorKogan LiorKogan changed the title master --> primary Minor edits Jun 28, 2026
Clarify the description of the `XIDMPRECORD` command by removing 'on an existing stream message'.
@LiorKogan LiorKogan requested a review from dwdougherty June 28, 2026 04:43
Updated the wording for clarity regarding IDMP duration retention.
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