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
sql: remove two private settings #49661
Conversation
Is it valid to delete cluster settings? I know there's weird stuff if you load a binary on a database that has cluster settings that don't exist anymore. I run into it if I make a binary with a new cluster setting, run the binary, then go to current master without the cluster setting and rerun on the same data directory - there will be a log message "unknown cluster setting blah". |
I've just tried creating a store on master and disabling these two settings, then restarting cockroach with this PR, looked at all cluster settings (and the settings were removed), then restarted with the master sha and the settings were correctly set to Couple of other settings that we could - but I'm less inclined to - remove are |
@dt Informed me on another PR about the retired settings list. |
Oh, thanks Peter. |
c3ab563
to
d46b02f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed 2 of 2 files at r1.
Reviewable status: complete! 1 of 0 LGTMs obtained
This commit removes two private settings that could disable planning of multiple join readers (i.e would disable distribution) and disable planning merge joiners (i.e. would simply prohibit the use of merge joiners). It also retires two settings that have been removed after 20.1 was cut. Release note: None
d46b02f
to
9d7d646
Compare
TFTR! bors r+ |
Build succeeded |
This commit removes two private settings that could disable planning of
multiple join readers (i.e would disable distribution) and disable
planning merge joiners (i.e. would simply prohibit the use of merge
joiners). It also retires two settings that have been removed after 20.1
was cut.
Release note: None