-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Addition of @@enable_system_settings to vtgate #7300
Conversation
Signed-off-by: Florent Poinsard <[email protected]>
Signed-off-by: Florent Poinsard <[email protected]>
The new variable is not tested for all test cases yet. Specifically the following one: vitess/go/vt/vtgate/executor_select_test.go Line 319 in 0fa0fc7
This specific test case relates to the issue mentioned in #7301. |
Signed-off-by: Florent Poinsard <[email protected]>
Signed-off-by: Andres Taylor <[email protected]>
Updated test
Signed-off-by: Florent Poinsard <[email protected]>
Signed-off-by: Florent Poinsard <[email protected]>
It would be great if you could add an end to end test in https://github.com/vitessio/vitess/blob/master/go/test/endtoend/vtgate/reservedconn/sysvar_test.go |
Signed-off-by: Florent Poinsard <[email protected]>
Signed-off-by: Florent Poinsard <[email protected]>
Signed-off-by: Florent Poinsard <[email protected]>
Really nice. Welcome as new contributor! 🤗 |
Awesome! Thank you 🎉 |
Signed-off-by: Florent Poinsard [email protected]
Description
This pull request introduces a new system variable:
enable_system_settings
. ThisVitessAware
variable is used to define wether the current session is allowed to use reserved connection or not.The default value of the variable is based on VTGate's
-enable_system_settings
flag.Fixes #6860.
Checklist
Impacted Areas in Vitess
Components that this PR will affect: