Skip to content

Add support for ZREMRANGEBYLEX command. #1968

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 5 commits into from
Closed

Add support for ZREMRANGEBYLEX command. #1968

wants to merge 5 commits into from

Conversation

@christophstrobl
Copy link
Member

@christophstrobl christophstrobl commented Feb 15, 2021

Closes: #1816

@christophstrobl
Prepare issue branch.
bef52ab
@christophstrobl
Preserve order in SetConverter.
31b3bf6 
This commit fixes an issue where the order of elements in a set is not preserved when converting elements of the set.
@christophstrobl
Add sync support for ZREMRANGEBYLEX.
b04c2ff
@christophstrobl
Add reactive support for ZREMRANGEBYLEX.
85c6f9f
@christophstrobl
Support ZREMRANGEBYLEX via Reactive-/ZSetOperations and RedisZSet.
3ef9d1e 
Also provide a Kotlin coroutines variant via the extension for ReactiveZSetOperations.
mp911de pushed a commit that referenced this pull request Feb 15, 2021
@christophstrobl @mp911de
Preserve order in SetConverter.
04707ec 
This commit fixes an issue where the order of elements in a set is not preserved when converting elements of the set.

Closes #1969.
Original pull request: #1968.
mp911de pushed a commit that referenced this pull request Feb 15, 2021
@christophstrobl @mp911de
Add support for ZREMRANGEBYLEX.
507a882 
We now support ZREMRANGEBYLEX for Synchronous, Reactive, ZSetOperations and RedisZSet.

Also provide a Kotlin coroutines variant via the extension for ReactiveZSetOperations.

Closes #1816.
Original pull request: #1968.
mp911de added a commit that referenced this pull request Feb 15, 2021
@mp911de
Polishing.
febdb82 
Reformat code. Tweak Javadoc

See #1816.
Original pull request: #1968.
mp911de pushed a commit that referenced this pull request Feb 15, 2021
@christophstrobl @mp911de
Preserve order in SetConverter.
d0e44f5 
This commit fixes an issue where the order of elements in a set is not preserved when converting elements of the set.

Closes #1969.
Original pull request: #1968.
mp911de pushed a commit that referenced this pull request Feb 15, 2021
@christophstrobl @mp911de
Preserve order in SetConverter.
206d87f 
This commit fixes an issue where the order of elements in a set is not preserved when converting elements of the set.

Closes #1969.
Original pull request: #1968.
@mp911de
Copy link
Member

mp911de commented Feb 15, 2021

That's merged and polished now. The SetConverter fix is backported to 2.4.x. and 2.3.x.

@mp911de mp911de closed this Feb 15, 2021
@mp911de mp911de deleted the issue/1816 branch February 15, 2021 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mp911de mp911de Awaiting requested review from mp911de

Assignees

@mp911de mp911de

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add support for removeRangeByLex in ZSetOperations [DATAREDIS-1244]

3 participants

@christophstrobl @mp911de