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

Cleanup MULTI state if MULTI command fails #1650

Closed
mp911de opened this issue Mar 8, 2021 · 0 comments
Closed

Cleanup MULTI state if MULTI command fails #1650

mp911de opened this issue Mar 8, 2021 · 0 comments
Labels
type: task A general task
Milestone
6.0.3

Comments

@mp911de
Copy link
Collaborator

mp911de commented Mar 8, 2021

We intend to activate MultiOutput before sending the actual MULTI command. This can however lead to a state where MultiOutput is set on the connection but the MULTI command was rejected or never sent. We should investigate what we can do here.

See also #1625.
@mp911de mp911de added the type: task A general task label Mar 8, 2021
@mp911de mp911de mentioned this issue Mar 8, 2021
Closed
mp911de added a commit that referenced this issue Mar 11, 2021
@mp911de
Cleanup MULTI state if MULTI command fails #1650
8adefb0 
We now clear the MULTI state on a best-effort basis when the MULTI command fails (Redis error, timeout, …).
mp911de added a commit that referenced this issue Mar 11, 2021
@mp911de
Cleanup MULTI state if MULTI command fails #1650
8410593 
We now clear the MULTI state on a best-effort basis when the MULTI command fails (Redis error, timeout, …).
@mp911de mp911de added this to the 6.0.3 milestone Mar 11, 2021
@mp911de mp911de closed this as completed Mar 11, 2021
This was referenced Mar 16, 2021
Closed
Closed
Merged
Merged
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: task A general task
Projects
None yet
Milestone
6.0.3
Development

No branches or pull requests
1 participant
@mp911de