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

twemproxy支持blpop指令么? #670

Open
wyl9527 opened this issue Aug 3, 2022 · 2 comments
Open

twemproxy支持blpop指令么? #670

wyl9527 opened this issue Aug 3, 2022 · 2 comments

Comments

@wyl9527
Copy link

wyl9527 commented Aug 3, 2022

文档显示twemrproxy不支持blpop指令,请问什么时候可以支持?谢谢

@yuerugou54
Copy link

yuerugou54 commented Aug 3, 2022 via email

@TysonAndre
Copy link
Collaborator

#519

It's very likely the decision to leave it out was deliberate on the part of the original maintainers - it was documented that the command is not supported.

If twemproxy has 200 clients and a small number of redis servers, and one of them issues a BRPOP or BLPOP, then 199 other clients will see an unexpected pause of an unknown length (for commands forwarded to the same server), possibly even seconds, causing an unexpected delay and adversely impacting the overall host/application performance, in a reason that those 199 clients have no visibility into.
...
As a workaround, it's already possible to call BLPOP/BRPOP in lua scripting (eval) if you're sure that your use case has timeouts properly configured or won't have adverse performance impact from blocking commands interfering with non-blocking commands.

This isn't currently planned.

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

No branches or pull requests

3 participants