Skip to content
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.

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

Add support for BLMPOP #2442

Merged
merged 3 commits into from
Feb 14, 2023
Merged

Add support for BLMPOP #2442

merged 3 commits into from
Feb 14, 2023

Conversation

yikuaibro
Copy link
Contributor

@monkey92t I'm learning from LMPOP, so I tried adding the BLMPOP command. Please help to review whether this PR meets the requirements of the BLMPOP command.
Fix: #2383

@monkey92t
Copy link
Collaborator

I am reflecting on whether the setting of ListElementCmd is correct, wait for a while.

@yikuaibro
Copy link
Contributor Author

I am reflecting on whether the setting of ListElementCmd is correct, wait for a while.

OK

@monkey92t
Copy link
Collaborator

@yikuaibro pull master, link #2443

@yikuaibro
Copy link
Contributor Author

@yikuaibro pull master, link #2443

Got it.

@yikuaibro
Copy link
Contributor Author

I have fixed it.

@monkey92t monkey92t merged commit bd0d1c2 into redis:master Feb 14, 2023
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 BLMPOP command
2 participants