-
Notifications
You must be signed in to change notification settings - Fork 998
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
MULTI does not allow SELECT #4120
Comments
Thanks. Do you know if database id in select can actually change during MULTI? |
|
No I dont think it can change. Well technically you can send a raw query but i dont think we need to worry about that. |
Fixes #4120 Signed-off-by: Roman Gershman <[email protected]>
Fixes #4120 Signed-off-by: Roman Gershman <[email protected]>
Fixes #4120 Signed-off-by: Roman Gershman <[email protected]>
Describe the bug
Im trying to use Dragonfly DB with StackExchange.Redis, which is the most popular redis library for C#.
However transactions do not work at all. After some debugging, I figured out the core issue. The issue is that dragonflydb doesnt seem to allow SELECT during MULTI. Dont ask me why it does that, but it does.
The command that SE.Redis sends is for example
To Reproduce in CLI
Steps to reproduce the behavior:
MULTI
SELECT 0
(You can already see the error message)EXEC
to see the final error message that the library also responds withExpected behavior
SELECT should be allowed in transactions, especially
SELECT 0
Screenshots
DragonFly logs
Wireshark capture
Environment (please complete the following information):
uname -a
Reproducible Code Snippet
Additional context
This also affects libraries that build upon StackExchange.Redis like Redis.OM, this is where I encountered the bug the first time.
The text was updated successfully, but these errors were encountered: