Skip to content

Commit 5016f37

Browse files
authored
Merge pull request ClickHouse#79037 from Algunenano/settings_trash
Place 25.4 setting changes under 25.4
2 parents e08e135 + ab5ea64 commit 5016f37

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

src/Core/SettingsChangesHistory.cpp

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -87,6 +87,11 @@ const VersionToSettingsChangesMap & getSettingsChangesHistory()
8787
{"query_condition_cache_store_conditions_as_plaintext", false, false, "New setting"},
8888
{"min_os_cpu_wait_time_ratio_to_throw", 0, 2, "New setting"},
8989
{"max_os_cpu_wait_time_ratio_to_throw", 0, 6, "New setting"},
90+
{"use_local_cache_for_remote_storage", true, false, "Obsolete setting."},
91+
{"iceberg_timestamp_ms", 0, 0, "New setting."},
92+
{"iceberg_snapshot_id", 0, 0, "New setting."},
93+
{"use_iceberg_metadata_files_cache", true, true, "New setting"},
94+
{"query_plan_join_shard_by_pk_ranges", false, false, "New setting"},
9095
/// Release closed. Please use 25.5
9196
});
9297
addSettingsChanges(settings_changes_history, "25.3",
@@ -101,14 +106,9 @@ const VersionToSettingsChangesMap & getSettingsChangesHistory()
101106
{"allow_experimental_database_unity_catalog", false, false, "Allow experimental database engine DataLakeCatalog with catalog_type = 'unity'"},
102107
{"allow_experimental_database_glue_catalog", false, false, "Allow experimental database engine DataLakeCatalog with catalog_type = 'glue'"},
103108
{"use_page_cache_with_distributed_cache", false, false, "New setting"},
104-
{"use_iceberg_metadata_files_cache", true, true, "New setting"},
105109
{"use_query_condition_cache", false, false, "New setting."},
106-
{"query_plan_join_shard_by_pk_ranges", false, false, "New setting"},
107-
{"iceberg_timestamp_ms", 0, 0, "New setting."},
108-
{"iceberg_snapshot_id", 0, 0, "New setting."},
109110
{"parallel_replicas_for_cluster_engines", false, true, "New setting."},
110111
/// Release closed. Please use 25.4
111-
{"use_local_cache_for_remote_storage", true, false, "Obsolete setting."},
112112
});
113113
addSettingsChanges(settings_changes_history, "25.2",
114114
{

tests/docker_scripts/upgrade_runner.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,7 @@ IS_SANITIZED=$(clickhouse-local --query "SELECT value LIKE '%-fsanitize=%' FROM
142142
if [ "${IS_SANITIZED}" -eq "0" ]
143143
then
144144
save_settings_clean 'new_settings.native'
145-
save_merge_tree_settings_clean 'new_merge_tree_settings.native'
145+
save_mergetree_settings_clean 'new_merge_tree_settings.native'
146146
clickhouse-local -nmq "
147147
CREATE TABLE old_settings AS file('old_settings.native');
148148
CREATE TABLE old_merge_tree_settings AS file('old_merge_tree_settings.native');
@@ -161,7 +161,7 @@ then
161161
SELECT arrayJoin(tupleElement(changes, 'name'))
162162
FROM
163163
(
164-
SELECT *, splitByChar('.', version) AS version_array FROM system.settings_changes WHERE type = 'Core'
164+
SELECT *, splitByChar('.', version) AS version_array FROM system.settings_changes WHERE type = 'Session'
165165
)
166166
WHERE (version_array[1]::UInt64 * 100 + version_array[2]::UInt64) > (SELECT v FROM old_version LIMIT 1)
167167
))
@@ -197,7 +197,7 @@ then
197197
SELECT arrayJoin(tupleElement(changes, 'name'))
198198
FROM
199199
(
200-
SELECT *, splitByChar('.', version) AS version_array FROM system.settings_changes WHERE type = 'Core'
200+
SELECT *, splitByChar('.', version) AS version_array FROM system.settings_changes WHERE type = 'Session'
201201
)
202202
WHERE (version_array[1]::UInt64 * 100 + version_array[2]::UInt64) > (SELECT v FROM old_version LIMIT 1)
203203
))

0 commit comments

Comments
 (0)