-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
Make Query Cache server-level settings reconfigurable at runtime #45758
Conversation
016fed6
to
9cb983d
Compare
9cb983d
to
540d672
Compare
540d672
to
5f70ac5
Compare
5f70ac5
to
216636f
Compare
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
programs/server/Server.cpp
Outdated
global_context->setQueryCache(); | ||
global_context->updateQueryCacheConfiguration(config()); |
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.
don't you think setQueryCache
is loosing its meaning here? maybe make it to accept config parameter right away?
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.
Semantics are a bit weird, I agree, thanks. Let me make it more pretty.
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.
overall LGTM, but consider my comment above
91b8e6e
to
b1889b0
Compare
+ rename setting
max_entry_records
tomax_entry_rows
for consistency with other settings.Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
The query cache's server-level settings are now reconfigurable at runtime