Skip to content

Added new stream commands#3450

Merged
htemelski-oss merged 6 commits into
masterfrom
x_command
Jul 31, 2025
Merged

Added new stream commands#3450
htemelski-oss merged 6 commits into
masterfrom
x_command

Conversation

@htemelski-oss

Copy link
Copy Markdown
Contributor

Added support for the new XACKDEL and XDELEX commands.
Extended the existing XADD and XTRIM coomands.

@ndyakov ndyakov left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@htemelski-redis thank you for the work here, I left some comments that need to be addressed before we can move forward with it.

Comment thread docker-compose.yml
Comment thread stream_commands.go Outdated
Comment thread stream_commands.go Outdated
Comment thread stream_commands.go

@ndyakov ndyakov left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

looks good to me

@htemelski-oss htemelski-oss merged commit bcdb7d1 into master Jul 31, 2025
20 checks passed
ofekshenawa pushed a commit that referenced this pull request Aug 10, 2025
* added new stream commands

* updated docker image

* fixed command return type

* fixed tests

* addressed PR comments

---------

Co-authored-by: Nedyalko Dyakov <1547186+ndyakov@users.noreply.github.com>
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