Skip to content

Commit d5710fc

Browse files
authored
Merge pull request #1084 from Altinity/fixes/25.8/fix_02995_new_settings_history
Test, settings, and configuration
2 parents befc762 + ab336b7 commit d5710fc

File tree

6 files changed

+21
-23
lines changed

6 files changed

+21
-23
lines changed

programs/server/config.xml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1733,9 +1733,9 @@
17331733
<!-- the ClickHouse core developers team. -->
17341734
<!-- Doing so at least in pre-production environments is highly appreciated -->
17351735
<!-- The reports are anonymized -->
1736-
<enabled>true</enabled>
1737-
<send_logical_errors>true</send_logical_errors>
1738-
<endpoint>https://crash.clickhouse.com/</endpoint>
1736+
<enabled>false</enabled>
1737+
<send_logical_errors>false</send_logical_errors>
1738+
<endpoint></endpoint>
17391739
</send_crash_reports>
17401740

17411741
<!-- Uncomment to disable ClickHouse internal DNS caching. -->

src/Core/Settings.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6645,13 +6645,13 @@ The timeout in milliseconds for connecting to a remote replica during query exec
66456645
DECLARE(Bool, parallel_replicas_for_cluster_engines, true, R"(
66466646
Replace table function engines with their -Cluster alternatives
66476647
)", 0) \
6648-
DECLARE_WITH_ALIAS(Bool, allow_experimental_database_iceberg, false, R"(
6648+
DECLARE_WITH_ALIAS(Bool, allow_experimental_database_iceberg, true, R"(
66496649
Allow experimental database engine DataLakeCatalog with catalog_type = 'iceberg'
66506650
)", BETA, allow_database_iceberg) \
6651-
DECLARE_WITH_ALIAS(Bool, allow_experimental_database_unity_catalog, false, R"(
6651+
DECLARE_WITH_ALIAS(Bool, allow_experimental_database_unity_catalog, true, R"(
66526652
Allow experimental database engine DataLakeCatalog with catalog_type = 'unity'
66536653
)", BETA, allow_database_unity_catalog) \
6654-
DECLARE_WITH_ALIAS(Bool, allow_experimental_database_glue_catalog, false, R"(
6654+
DECLARE_WITH_ALIAS(Bool, allow_experimental_database_glue_catalog, true, R"(
66556655
Allow experimental database engine DataLakeCatalog with catalog_type = 'glue'
66566656
)", BETA, allow_database_glue_catalog) \
66576657
DECLARE_WITH_ALIAS(Bool, allow_experimental_analyzer, true, R"(
@@ -7083,7 +7083,7 @@ Execute request to object storage as remote on one of object_storage_cluster nod
70837083
DECLARE_WITH_ALIAS(Bool, allow_experimental_time_series_aggregate_functions, false, R"(
70847084
Experimental timeSeries* aggregate functions for Prometheus-like timeseries resampling, rate, delta calculation.
70857085
)", EXPERIMENTAL, allow_experimental_ts_to_grid_aggregate_function) \
7086-
DECLARE_WITH_ALIAS(Bool, allow_experimental_export_merge_tree_part, false, R"(
7086+
DECLARE_WITH_ALIAS(Bool, allow_experimental_export_merge_tree_part, true, R"(
70877087
Experimental export merge tree part.
70887088
)", EXPERIMENTAL, allow_experimental_export_merge_tree_part) \
70897089
\

src/Core/SettingsChangesHistory.cpp

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -44,10 +44,9 @@ const VersionToSettingsChangesMap & getSettingsChangesHistory()
4444
{"allow_experimental_iceberg_read_optimization", true, true, "New setting."},
4545
{"object_storage_cluster_join_mode", "allow", "allow", "New setting"},
4646
{"lock_object_storage_task_distribution_ms", 500, 500, "Raised the value to 500 to avoid hoping tasks between executors."},
47-
{"object_storage_cluster", "", "", "New setting"},
48-
{"object_storage_max_nodes", 0, 0, "New setting"},
4947
{"allow_retries_in_cluster_requests", false, false, "New setting"},
5048
{"object_storage_remote_initiator", false, false, "New setting."},
49+
{"allow_experimental_export_merge_tree_part", false, true, "Turned ON by default for Antalya."},
5150
});
5251
addSettingsChanges(settings_changes_history, "25.8",
5352
{
@@ -143,13 +142,6 @@ const VersionToSettingsChangesMap & getSettingsChangesHistory()
143142
{"distributed_plan_force_shuffle_aggregation", 0, 0, "New experimental setting"},
144143
{"allow_experimental_insert_into_iceberg", false, false, "New setting."},
145144
/// RELEASE CLOSED
146-
{"allow_experimental_database_iceberg", false, true, "Turned ON by default for Antalya"},
147-
{"allow_experimental_database_unity_catalog", false, true, "Turned ON by default for Antalya"},
148-
{"allow_experimental_database_glue_catalog", false, true, "Turned ON by default for Antalya"},
149-
{"output_format_parquet_enum_as_byte_array", true, true, "Enable writing Enum as byte array in Parquet by default"},
150-
{"lock_object_storage_task_distribution_ms", 0, 0, "New setting."},
151-
{"object_storage_cluster", "", "", "New setting"},
152-
{"object_storage_max_nodes", 0, 0, "New setting"},
153145
});
154146
addSettingsChanges(settings_changes_history, "25.6.5.2000",
155147
{
@@ -161,6 +153,7 @@ const VersionToSettingsChangesMap & getSettingsChangesHistory()
161153
{"object_storage_max_nodes", 0, 0, "New setting"},
162154
{"allow_experimental_export_merge_tree_part", false, false, "New setting."},
163155
{"export_merge_tree_part_overwrite_file_if_exists", false, false, "New setting."},
156+
{"lock_object_storage_task_distribution_ms", 0, 0, "New setting."},
164157
});
165158
addSettingsChanges(settings_changes_history, "25.6",
166159
{

tests/queries/0_stateless/02995_new_settings_history.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,8 @@ fi
2323

2424
# Note that this is a broad check. A per version check is done in the upgrade test
2525
# Baselines generated with v25.7.1 (pre-release)
26-
# clickhouse local --query "select name, default from system.settings order by name format TSV" > 02995_settings_25_7_1.tsv
27-
# clickhouse local --query "select name, value from system.merge_tree_settings order by name format TSV" > 02995_merge_tree_settings_settings_25_7_1.tsv
26+
# clickhouse local --query "select name, default from system.settings WHERE NOT has((select flatten(groupArray(changes.name)) from system.settings_changes where version >= '25.8'), name) order by name format TSV" > 02995_settings_25_7_1.tsv
27+
# clickhouse local --query "select name, value from system.merge_tree_settings WHERE NOT has((select flatten(groupArray(changes.name)) from system.settings_changes where version >= '25.8'), name) order by name format TSV" > 02995_merge_tree_settings_settings_25_7_1.tsv
2828
$CLICKHOUSE_LOCAL --query "
2929
WITH old_settings AS
3030
(

tests/queries/0_stateless/02995_settings_25_7_1.tsv

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -26,15 +26,16 @@ allow_experimental_bigint_types 1
2626
allow_experimental_codecs 0
2727
allow_experimental_correlated_subqueries 0
2828
allow_experimental_database_atomic 1
29-
allow_experimental_database_glue_catalog 0
29+
allow_experimental_database_glue_catalog 1
3030
allow_experimental_database_hms_catalog 0
31-
allow_experimental_database_iceberg 0
31+
allow_experimental_database_iceberg 1
3232
allow_experimental_database_materialized_mysql 0
3333
allow_experimental_database_materialized_postgresql 0
3434
allow_experimental_database_replicated 1
35-
allow_experimental_database_unity_catalog 0
35+
allow_experimental_database_unity_catalog 1
3636
allow_experimental_delta_kernel_rs 1
3737
allow_experimental_dynamic_type 1
38+
allow_experimental_export_merge_tree_part 0
3839
allow_experimental_full_text_index 0
3940
allow_experimental_funnel_functions 0
4041
allow_experimental_geo_types 1
@@ -360,6 +361,7 @@ engine_url_skip_empty_files 0
360361
errors_output_format CSV
361362
exact_rows_before_limit 0
362363
except_default_mode ALL
364+
export_merge_tree_part_overwrite_file_if_exists 0
363365
external_storage_connect_timeout_sec 10
364366
external_storage_max_read_bytes 0
365367
external_storage_max_read_rows 0
@@ -584,13 +586,13 @@ input_format_parquet_case_insensitive_column_matching 0
584586
input_format_parquet_enable_json_parsing 1
585587
input_format_parquet_enable_row_group_prefetch 1
586588
input_format_parquet_filter_push_down 1
587-
input_format_parquet_use_metadata_cache 1
588589
input_format_parquet_import_nested 0
589590
input_format_parquet_local_file_min_bytes_for_seek 8192
590591
input_format_parquet_max_block_size 65409
591592
input_format_parquet_prefer_block_bytes 16744704
592593
input_format_parquet_preserve_order 0
593594
input_format_parquet_skip_columns_with_unsupported_types_in_schema_inference 0
595+
input_format_parquet_use_metadata_cache 1
594596
input_format_parquet_use_native_reader 0
595597
input_format_protobuf_flatten_google_wrappers 0
596598
input_format_protobuf_skip_fields_with_unsupported_types_in_schema_inference 0
@@ -863,6 +865,8 @@ network_zstd_compression_level 1
863865
normalize_function_names 1
864866
number_of_mutations_to_delay 0
865867
number_of_mutations_to_throw 0
868+
object_storage_cluster
869+
object_storage_max_nodes 0
866870
odbc_bridge_connection_pool_size 16
867871
odbc_bridge_use_connection_pooling 1
868872
odbc_max_field_size 0

tests/queries/0_stateless/03413_experimental_settings_cannot_be_enabled_by_default.sql

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
SELECT name, value FROM system.settings WHERE tier = 'Experimental' AND type = 'Bool' AND value != '0' AND name NOT IN (
88
'throw_on_unsupported_query_inside_transaction',
99
-- turned ON for Altinity Antalya builds specifically
10-
'allow_experimental_iceberg_read_optimization'
10+
'allow_experimental_iceberg_read_optimization',
11+
'allow_experimental_export_merge_tree_part',
1112
);
1213
SELECT name, value FROM system.merge_tree_settings WHERE tier = 'Experimental' AND type = 'Bool' AND value != '0' AND name NOT IN ('remove_rolled_back_parts_immediately');

0 commit comments

Comments
 (0)