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.

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

redis API: support brpop, blpop #114

Closed
jasonhe88 opened this issue Jun 25, 2017 · 8 comments · Fixed by #1548
Closed

redis API: support brpop, blpop #114

jasonhe88 opened this issue Jun 25, 2017 · 8 comments · Fixed by #1548
Labels
enhancement

Comments

@jasonhe88
Copy link

jasonhe88 commented Jun 25, 2017 •
edited by wanghenshui
Loading

请问未来有没有支持 brpop, blpop 的计划？，

谢谢！
@baotiao
Copy link
Contributor

baotiao commented Jun 26, 2017

会有的.

@jasonhe88
Copy link
Author

太好了，谢谢

@Hisoka-J
Copy link
Collaborator

你将要用在什么场景？

AlexStocks pushed a commit to ipixiu/pika that referenced this issue Dec 21, 2018
@Axlgrep
use the cpplint.py specification code format (OpenAtomFoundation#114)
3715974
@jellychen
Copy link

so 什么时候支持啊？

@wanghenshui wanghenshui changed the title 请问未来有没有支持 brpop, blpop 的计划？ redis API: support brpop, blpop Feb 22, 2023
@wanghenshui wanghenshui mentioned this issue Feb 22, 2023
Closed
78 tasks
@Tangruilin
Copy link
Collaborator

这个我可以考虑进行支持，这个是需要看 redis 这块是怎么做的吗？

@AlexStocks
Copy link
Contributor

@Tangruilin I have assigned this issue to you.

@AlexStocks
Copy link
Contributor

AlexStocks commented May 6, 2023 •
edited
Loading

@cheniujh chenjunhua

@yaoyinnan yaoyinnan mentioned this issue Jun 30, 2023
Merged
12 tasks
@AlexStocks
Copy link
Contributor 

        * 0228 @Tangruilin 
        * 0506 @cheniujh 
        * 0520 俊华在写，继续推进
        * 0603 提了PR https://github.com/OpenAtomFoundation/pika/pull/1548 待CR
        * 0610 RpopLpush 存在并发的问题：https://github.com/OpenAtomFoundation/pika/issues/1608
        * 0617 1.brpop/blpop的pr缺review(目前只有水哥review过），rpoplpush的同步问题已经提了pr
        * 0624 解决主从同步单侧后合入
        * 0701 主从单测问题解决后这个就可以合并了

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: support blpop/brpop cheniujh/pika
7 participants
@baotiao @jellychen @jasonhe88 @Leviathan1995 @AlexStocks @Hisoka-J @Tangruilin