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
Ulid short circuit #60211
Ulid short circuit #60211
Conversation
This is an automated comment for commit 28eb159 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
|
a14bb49
to
af14343
Compare
@@ -0,0 +1,4 @@ | |||
-- Tags: no-fasttest | |||
|
|||
SELECT if(length(x) = 26, ULIDStringToDateTime(x, 'Europe/Madrid'), toDateTime('2024-02-21 12:00:00')) AS datetime |
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.
toDateTime
probably needs a timezone too: https://s3.amazonaws.com/clickhouse-test-reports/60211/af14343462b10828fc0e6b9eac03590a42956546/stateless_tests__aarch64_.html
2024-02-21 19:41:28 --- /usr/share/clickhouse-test/queries/0_stateless/02995_ulid_short_circuit.reference 2024-02-21 19:32:56.232416132 +0300
2024-02-21 19:41:28 +++ /tmp/clickhouse-test/0_stateless/02995_ulid_short_circuit.stdout 2024-02-21 19:41:28.112429839 +0300
2024-02-21 19:41:28 @@ -1,2 +1,2 @@
2024-02-21 19:41:28 2024-02-20 16:53:57.105
2024-02-21 19:41:28 -2024-02-21 12:00:00.000
2024-02-21 19:41:28 +2024-02-21 10:00:00.000
2024-02-21 19:41:28
2024-02-21 19:41:28
2024-02-21 19:41:28 Settings used in the test: --max_insert_threads 0 --group_by_two_level_threshold 1000000 --group_by_two_level_threshold_bytes 30666637 --distributed_aggregation_memory_efficient 1 --fsync_metadata 1 --output_format_parallel_formatting 0 --input_format_parallel_parsing 0 --min_chunk_bytes_for_parallel_parsing 2967097 --max_read_buffer_size 1022899 --prefer_localhost_replica 0 --max_block_size 40966 --max_threads 33 --optimize_append_index 1 --optimize_if_chain_to_multiif 0 --optimize_if_transform_strings_to_enum 0 --optimize_read_in_order 1 --optimize_or_like_chain 1 --optimize_substitute_columns 0 --enable_multiple_prewhere_read_steps 1 --read_in_order_two_level_merge_threshold 16 --optimize_aggregation_in_order 0 --aggregation_in_order_max_block_bytes 22531778 --min_compress_block_size 2418609 --max_compress_block_size 1941249 --use_uncompressed_cache 1 --min_bytes_to_use_direct_io 5924730389 --min_bytes_to_use_mmap_io 1 --local_filesystem_read_method io_uring --remote_filesystem_read_method threadpool --local_filesystem_read_prefetch 1 --filesystem_cache_segments_batch_size 10 --read_from_filesystem_cache_if_exists_otherwise_bypass_cache 0 --throw_on_error_from_cache_on_write_operations 0 --remote_filesystem_read_prefetch 0 --allow_prefetched_read_pool_for_remote_filesystem 1 --filesystem_prefetch_max_memory_usage 128Mi --filesystem_prefetches_limit 0 --filesystem_prefetch_min_bytes_for_single_read_task 1Mi --filesystem_prefetch_step_marks 50 --filesystem_prefetch_step_bytes 100Mi --compile_aggregate_expressions 0 --compile_sort_description 0 --merge_tree_coarse_index_granularity 31 --optimize_distinct_in_order 0 --max_bytes_before_external_sort 10737418240 --max_bytes_before_external_group_by 8831025856 --max_bytes_before_remerge_sort 347721009 --optimize_sorting_by_input_stream_properties 0 --http_response_buffer_size 7932347 --http_wait_end_of_query True --enable_memory_bound_merging_of_aggregation_results 1 --min_count_to_compile_expression 3 --min_count_to_compile_aggregate_expression 3 --min_count_to_compile_sort_description 0 --session_timezone Asia/Qatar
2024-02-21 19:41:28
2024-02-21 19:41:28 MergeTree settings used in test: --ratio_of_defaults_for_sparse_serialization 1.0 --prefer_fetch_merged_part_size_threshold 10737418240 --vertical_merge_algorithm_min_rows_to_activate 1000000 --vertical_merge_algorithm_min_columns_to_activate 100 --allow_vertical_merges_from_compact_to_wide_parts 0 --min_merge_bytes_to_use_direct_io 10737418240 --index_granularity_bytes 25180644 --merge_max_block_size 1348 --index_granularity 45527 --min_bytes_for_wide_part 1073741824 --marks_compress_block_size 78946 --primary_key_compress_block_size 71537 --replace_long_file_name_to_hash 1 --max_file_name_length 120
2024-02-21 19:41:28
2024-02-21 19:41:28 Database: test_vtf20tvf
e20d0e6
to
3a65272
Compare
Looks like you were hit by the io_uring issue in the CI (see #60373). Rebase with master. |
e4c1767
to
28eb159
Compare
@Algunenano is there anything I can do for the failed Bugfix validation? Still two queued jobs but mostly everything passed. |
It seems the check itself is broken. because the test do reproduce the issue. Nevertheless, I'm switching the PR to improvement since short circuit on functions is a feature |
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Short circuit execution for
ULIDStringToDateTime