-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Speed up connections by sending SetInfo via a pipeline #2880
Conversation
part of #2850 |
b1f20d1
to
7f4c593
Compare
@ofekshenawa @chayim This change will break users using Redis version < 7.2 which doesn't support the BTW, I think it'd be better to set |
Hello! This was released in a minor update release, but it would break all our apps, since we are using KeyDB and as far as I see it does not support Looking at the references to this PR, we are not the only ones having issues. Are there plans for fixing this or workarounds available? |
Same for me, I think there is a regression or a breaking change. I run into this error while calling redis |
Follow #2911 if you've found your way here because this change is breaking your apps. |
No description provided.