Skip to content

Conversation

b1ron
Copy link
Contributor

@b1ron b1ron commented May 29, 2024

Closes #2921.

@b1ron
Copy link
Contributor Author

b1ron commented May 29, 2024

Tests are failing because the argument is not yet supported.

@ofekshenawa ofekshenawa merged commit 3975cd5 into redis:master Jun 20, 2024
ofekshenawa pushed a commit that referenced this pull request Jun 20, 2024
* add support for XREAD last entry

* handle reading from multiple streams

* add test to ensure we block for empty stream

* small tweak

* add an option to XReadArgs instead

* modify test comment

* small preallocation optimization

* Changed argument to generic ID, skip tests on Enterprise

* Fix test case

* Updated expiration command

---------

Co-authored-by: Vladyslav Vildanov <117659936+vladvildanov@users.noreply.github.com>
Co-authored-by: vladvildanov <divinez122@outlook.com>
@b1ron b1ron deleted the xread-last-entry branch June 20, 2024 11:29
ofekshenawa pushed a commit that referenced this pull request Aug 10, 2025
* add support for XREAD last entry

* handle reading from multiple streams

* add test to ensure we block for empty stream

* small tweak

* add an option to XReadArgs instead

* modify test comment

* small preallocation optimization

* Changed argument to generic ID, skip tests on Enterprise

* Fix test case

* Updated expiration command

---------

Co-authored-by: Vladyslav Vildanov <117659936+vladvildanov@users.noreply.github.com>
Co-authored-by: vladvildanov <divinez122@outlook.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.

Support XREAD, XREADGROUP reading from last message of stream
3 participants