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
Allow max_size_to_drop settings in query time #57452
Allow max_size_to_drop settings in query time #57452
Conversation
This is an automated comment for commit 1489fc2 with description of existing statuses. It's updated for the latest CI running ❌ Click here to open a full report in a separate page Successful checks
|
283e7c7
to
f3a71c2
Compare
CI failures are unrelated:
|
Not only, it can be also used to prohibit removal of big "production" tables, and making it a user setting looks slightly unsafe... Though now ClickHouse support grants, but maybe it is OK to make it query setting not a table setting (like in #56473), since I guess prohibiting DROP explicitly now is the correct way (via GRANT/REVOKE). |
@azat it's OK to prevent users from removing tables by mistake. The thing is that now you have a few options to remove a table that you're sure you want to remove, you have to either increase the server setting, or create the file flag which requires SSH access to the instance. I'd prefer the query setting over the merge tree setting but I don't care that much for which one we decided to go if we end up with an easier way to drop/truncate tables we are sure we want to remove. By changes in #57638, looks like this setting is already a query setting in their internal fork. @nikitamikhaylov, what do you think? |
@jrdi Indeed. Let's just resolve conflicts in this PR and merge it. |
e5df1a9
to
463e082
Compare
bd9f049
to
8778d84
Compare
239815a
to
b391275
Compare
A bunch of stateless tests are failing with "Tests are not finished" is this something new? I can't see how the changes in this PR can cause this. |
b391275
to
1489fc2
Compare
Rebasing from master again since looks like the issues with stateless test have been fixed here #57974 |
Both seem unrelated. Checking https://s3.amazonaws.com/clickhouse-test-reports/57452/1489fc2bf74cd5e28f92ffc997a4713f662c3e48/upgrade_check__asan_/test_results.tsv looks like some tables can't be load due to issues with table engine. Probably related to #58026 |
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Allow to overwrite
max_partition_size_to_drop
andmax_table_size_to_drop
server settings in query time.Having those settings on the server side is good to avoid removing tables that are too big by mistake. But sometimes users want to remove them for real, and there is no easy way.