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

Close connection on session consistency loss by default #5855

Open
achamayou opened this issue Dec 6, 2023 · 0 comments
Open

Close connection on session consistency loss by default #5855

achamayou opened this issue Dec 6, 2023 · 0 comments

Comments

@achamayou
Copy link
Member

achamayou commented Dec 6, 2023

#4595 introduced error messages on session consistency breaks, followed by a session closure. Sessions can be long lived however, and users may not require session consistency (either because the application is idempotent, or because they check commit explicitly anyway).

In those cases, they would prefer an early session closure, resulting in one less (potentially confusing when it's delayed) error message.

Based on this feedback, I think we should at least change the default behaviour, and perhaps make this configurable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant