Skip to content

feat: add support for Redis Streams XINFO STREAM IDMP fields#965

Merged
rueian merged 1 commit intoredis:mainfrom
jinbum-kim:feat/rueidiscompat-xinfo-stream-idmp
Mar 4, 2026
Merged

feat: add support for Redis Streams XINFO STREAM IDMP fields#965
rueian merged 1 commit intoredis:mainfrom
jinbum-kim:feat/rueidiscompat-xinfo-stream-idmp

Conversation

@jinbum-kim
Copy link
Copy Markdown
Contributor

@jinbum-kim jinbum-kim commented Mar 3, 2026

closes #955

Port redis/go-redis#3685 XINFO STREAM IDMP fields to rueidiscompat.

  • Add IDMPDuration, IDMPMaxSize, PIDsTracked, IIDsTracked, IIDsAdded, IIDsDuplicates fields to XInfoStream and XInfoStreamFull structs
  • Parse the corresponding keys in XInfoStreamCmd.from() and XInfoStreamFullCmd.from()
  • Add XINFO STREAM / XINFO STREAM FULL tests in testAdapterRedis86() to verify new fields on Redis 8.6+

Note

Low Risk
Low risk: adds new optional fields to XInfoStream/XInfoStreamFull and parses extra map keys, with tests updated to account for the additional data returned by newer Redis versions.

Overview
Adds support for Redis Streams idempotent production (IDMP) metadata in XINFO STREAM and XINFO STREAM FULL responses.

XInfoStream and XInfoStreamFull now expose IDMPDuration, IDMPMaxSize, PIDsTracked, IIDsTracked, IIDsAdded, and IIDsDuplicates, and the corresponding command decoders populate these fields when present. Tests are updated to ignore these fields in existing equality assertions and add new coverage verifying the IDMP values are returned and parsed.

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

Signed-off-by: jinbum-kim <jinbum9958@gmail.com>
@jinbum-kim
Copy link
Copy Markdown
Contributor Author

Hello @rueian, PTAL! Thank you 🙂

@rueian rueian merged commit 98f32bd into redis:main Mar 4, 2026
28 checks passed
@jinbum-kim jinbum-kim deleted the feat/rueidiscompat-xinfo-stream-idmp branch March 5, 2026 01:07
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.

Add support for Redis Streams XINFO STREAM IDMP fields to rueidiscompat

2 participants