Skip to content

Conversation

@TropicolX
Copy link
Contributor

@TropicolX TropicolX commented Oct 3, 2025

Removed unnecessary details section and streamlined explanations about offset tracking and command processing.


Note

Streamlines the replication stage doc with a concise ACK/offset explanation, normalized examples, and an explicit RESP array response requirement.

  • Docs (stage_descriptions/replication-15-yd3.md):
    • Add an ACKs recap and replace the collapsible details with concise offset-tracking guidance, emphasizing that the current REPLCONF GETACK * is excluded.
    • Revise examples/tests: normalize command casing (REPLCONF, PING, SET), streamline comments, and update byte-offset calculations.
    • Clarify that replicas count bytes for all propagated commands, and keep the PING behavior note.
    • Add an explicit requirement that REPLCONF ACK <offset> be returned as a RESP array; remove redundant/extraneous notes.

Written by Cursor Bugbot for commit 96cf25c. This will update automatically on new commits. Configure here.

Removed unnecessary details section and streamlined explanations about offset tracking and command processing.
Clarify the offset tracking process for REPLCONF GETACK commands, including examples and expected responses.
Updated command syntax and clarified offset response requirements.
@TropicolX TropicolX marked this pull request as ready for review October 4, 2025 09:00
@TropicolX TropicolX merged commit 003dfb1 into main Oct 4, 2025
28 checks passed
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.

2 participants