Skip to content

Fix flaky 00906_low_cardinality_cache#92009

Merged
fm4v merged 2 commits intomasterfrom
fix-flaky-00906_low_cardinality_cache
Dec 12, 2025
Merged

Fix flaky 00906_low_cardinality_cache#92009
fm4v merged 2 commits intomasterfrom
fix-flaky-00906_low_cardinality_cache

Conversation

@fm4v
Copy link
Copy Markdown
Member

@fm4v fm4v commented Dec 11, 2025 •
edited
Loading

Close #91024
CIDB

Test failing with error

2025-12-11 17:43:56 Reason: return code:  160
2025-12-11 17:43:56 [d5a9c726ee24] 2025.12.11 18:43:42.531310 [ 1074893 ] {ac8c8ad8-4936-4da8-b2d8-bb89c4099753} <Error> executeQuery: Code: 160. DB::Exception: Estimated query execution time (778.76271 seconds) is too long. Maximum: 600. Estimated rows to process: 100000000 (38535385 read in 300.09921 seconds): While executing NumbersRange. (TOO_SLOW) (version 25.12.1.450) (from [::1]:37188) (comment: 00906_low_cardinality_cache.sql-test_u9huuo15) (query 4, line 6) (in query: insert into lc_00906 select '0123456789' from numbers(100000000);), Stack trace (when copying this message, always include the lines below):
2025-12-11 17:43:56 
2025-12-11 17:43:56 0. ./contrib/llvm-project/libcxx/include/__exception/exception.h:113: Poco::Exception::Exception(String const&, int) @ 0x000000002e98bee0
2025-12-11 17:43:56 1. ./ci/tmp/build/./src/Common/Exception.cpp:131: DB::Exception::Exception(DB::Exception::MessageMasked&&, int, bool) @ 0x0000000015fb52b4
2025-12-11 17:43:56 2. DB::Exception::Exception(String&&, int, String, bool) @ 0x0000000009b94c2e
2025-12-11 17:43:56 3. DB::Exception::Exception(PreformattedMessage&&, int) @ 0x0000000009b944c9
2025-12-11 17:43:56 4. ./src/Common/Exception.h:141: DB::Exception::Exception<double&, int, unsigned long&, unsigned long&, double&>(int, FormatStringHelperImpl<std::type_identity<double&>::type, std::type_identity<int>::type, std::type_identity<unsigned long&>::type, std::type_identity<unsigned long&>::type, std::type_identity<double&>::type>, double&, int&&, unsigned long&, unsigned long&, double&) @ 0x000000001e09635c
2025-12-11 17:43:56 5. ./ci/tmp/build/./src/QueryPipeline/ExecutionSpeedLimits.cpp:87: DB::ExecutionSpeedLimits::throttle(unsigned long, unsigned long, unsigned long, unsigned long, DB::OverflowMode) const @ 0x000000001e095e8a
2025-12-11 17:43:56 6. ./ci/tmp/build/./src/QueryPipeline/ReadProgressCallback.cpp:107: DB::ReadProgressCallback::onProgress(unsigned long, unsigned long, std::list<DB::StorageLimits, std::allocator<DB::StorageLimits>> const&) @ 0x000000001e0953c1
2025-12-11 17:43:56 7. ./ci/tmp/build/./src/Processors/Executors/ExecutionThreadContext.cpp:68: DB::ExecutionThreadContext::executeTask() @ 0x00000000262dd33a
2025-12-11 17:43:56 8. ./ci/tmp/build/./src/Processors/Executors/PipelineExecutor.cpp:351: DB::PipelineExecutor::executeStepImpl(unsigned long, DB::IAcquiredSlot*, std::atomic<bool>*) @ 0x00000000262c95d9
2025-12-11 17:43:56 9. ./ci/tmp/build/./src/Processors/Executors/PipelineExecutor.cpp:279: DB::PipelineExecutor::executeImpl(unsigned long, bool) @ 0x00000000262c879e
2025-12-11 17:43:56 10. ./ci/tmp/build/./src/Processors/Executors/PipelineExecutor.cpp:136: DB::PipelineExecutor::execute(unsigned long, bool) @ 0x00000000262c81ea
2025-12-11 17:43:56 11. ./ci/tmp/build/./src/Processors/Executors/CompletedPipelineExecutor.cpp:42: void std::__function::__policy_func<void ()>::__call_func[abi:ne210105]<ThreadFromGlobalPoolImpl<true, true>::ThreadFromGlobalPoolImpl<DB::CompletedPipelineExecutor::execute()::$_0>(DB::CompletedPipelineExecutor::execute()::$_0&&)::'lambda'()>(std::__function::__policy_storage const*) @ 0x00000000262c6db2
2025-12-11 17:43:56 12. ./contrib/llvm-project/libcxx/include/__functional/function.h:508: ? @ 0x0000000016183131
2025-12-11 17:43:56 13. ./contrib/llvm-project/libcxx/include/__type_traits/invoke.h:0: void* std::__thread_proxy[abi:ne210105]<std::tuple<std::unique_ptr<std::__thread_struct, std::default_delete<std::__thread_struct>>, void (ThreadPoolImpl<std::thread>::ThreadFromThreadPool::*)(), ThreadPoolImpl<std::thread>::ThreadFromThreadPool*>>(void*) @ 0x000000001618d21c
2025-12-11 17:43:56 14. __tsan_thread_start_func @ 0x0000000009affab8
2025-12-11 17:43:56 15. ? @ 0x0000000000094ac3
2025-12-11 17:43:56 16. ? @ 0x00000000001268c0
2025-12-11 17:43:56 
2025-12-11 17:43:56 Received exception from server (version 25.12.1):
2025-12-11 17:43:56 Code: 160. DB::Exception: Received from localhost:9000. DB::Exception: Estimated query execution time (778.76271 seconds) is too long. Maximum: 600. Estimated rows to process: 100000000 (38535385 read in 300.09921 seconds): While executing NumbersRange. (TOO_SLOW)
2025-12-11 17:43:56 (query: insert into lc_00906 select '0123456789' from numbers(100000000);)

I've analyzed 10 reports and found out that it fail most of the time when settings max_threads, max_insert_threads and/or vertical_merge_algorithm_min_rows_to_activate set to 1.

Changelog category (leave one):

  • Build/Testing/Packaging Improvement

Changelog entry (a user-readable short description of the changes that goes into CHANGELOG.md):

Fix flaky 00906_low_cardinality_cache

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

@fm4v fm4v enabled auto-merge December 11, 2025 22:50
@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Dec 11, 2025 •
edited
Loading

Workflow [PR], commit [21eb1bd]

Summary: ✅

@clickhouse-gh clickhouse-gh bot added the pr-build Pull request with build/testing/packaging improvement label Dec 11, 2025
@fm4v fm4v added the pr-must-backport Pull request should be backported intentionally. Use this label with great care! label Dec 11, 2025
@fm4v fm4v added this pull request to the merge queue Dec 12, 2025
Merged via the queue into master with commit 4631002 Dec 12, 2025
130 checks passed
@fm4v fm4v deleted the fix-flaky-00906_low_cardinality_cache branch December 12, 2025 01:29
@robot-ch-test-poll1 robot-ch-test-poll1 added the pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR label Dec 12, 2025
@robot-clickhouse robot-clickhouse mentioned this pull request Dec 12, 2025
Merged
robot-clickhouse added a commit that referenced this pull request Dec 12, 2025
@robot-clickhouse
Merge pull request #92012 from ClickHouse/cherrypick/25.3/92009
903657c 
Cherry pick #92009 to 25.3: Fix flaky 00906_low_cardinality_cache
robot-clickhouse added a commit that referenced this pull request Dec 12, 2025
@robot-clickhouse
Backport #92009 to 25.3: Fix flaky 00906_low_cardinality_cache
f365c89
This was referenced Dec 12, 2025
Merged
Merged
robot-clickhouse added a commit that referenced this pull request Dec 12, 2025
@robot-clickhouse
Merge pull request #92014 from ClickHouse/cherrypick/25.8/92009
0b44a11 
Cherry pick #92009 to 25.8: Fix flaky 00906_low_cardinality_cache
robot-clickhouse added a commit that referenced this pull request Dec 12, 2025
@robot-clickhouse
Backport #92009 to 25.8: Fix flaky 00906_low_cardinality_cache
eb3e751
This was referenced Dec 12, 2025
Merged
Merged
robot-clickhouse added a commit that referenced this pull request Dec 12, 2025
@robot-clickhouse
Merge pull request #92016 from ClickHouse/cherrypick/25.9/92009
a1e1e93 
Cherry pick #92009 to 25.9: Fix flaky 00906_low_cardinality_cache
robot-clickhouse added a commit that referenced this pull request Dec 12, 2025
@robot-clickhouse
Backport #92009 to 25.9: Fix flaky 00906_low_cardinality_cache
9a4a96b
This was referenced Dec 12, 2025
Merged
Merged
robot-clickhouse added a commit that referenced this pull request Dec 12, 2025
@robot-clickhouse
Merge pull request #92018 from ClickHouse/cherrypick/25.10/92009
efe080b 
Cherry pick #92009 to 25.10: Fix flaky 00906_low_cardinality_cache
robot-clickhouse added a commit that referenced this pull request Dec 12, 2025
@robot-clickhouse
Backport #92009 to 25.10: Fix flaky 00906_low_cardinality_cache
1b68115
This was referenced Dec 12, 2025
Merged
Merged
robot-clickhouse added a commit that referenced this pull request Dec 12, 2025
@robot-clickhouse
Merge pull request #92020 from ClickHouse/cherrypick/25.11/92009
93db048 
Cherry pick #92009 to 25.11: Fix flaky 00906_low_cardinality_cache
robot-clickhouse added a commit that referenced this pull request Dec 12, 2025
@robot-clickhouse
Backport #92009 to 25.11: Fix flaky 00906_low_cardinality_cache
9291462
@robot-clickhouse robot-clickhouse mentioned this pull request Dec 12, 2025
Merged
@robot-ch-test-poll4 robot-ch-test-poll4 added the pr-synced-to-cloud The PR is synced to the cloud repo label Dec 12, 2025
@robot-ch-test-poll1 robot-ch-test-poll1 added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Dec 12, 2025
clickhouse-gh bot added a commit that referenced this pull request Dec 12, 2025
@clickhouse-gh
Merge pull request #92015 from ClickHouse/backport/25.8/92009
42c88e6 
Backport #92009 to 25.8: Fix flaky 00906_low_cardinality_cache
clickhouse-gh bot added a commit that referenced this pull request Dec 12, 2025
@clickhouse-gh
Merge pull request #92017 from ClickHouse/backport/25.9/92009
be3aa84 
Backport #92009 to 25.9: Fix flaky 00906_low_cardinality_cache
clickhouse-gh bot added a commit that referenced this pull request Dec 12, 2025
@clickhouse-gh
Merge pull request #92019 from ClickHouse/backport/25.10/92009
f986356 
Backport #92009 to 25.10: Fix flaky 00906_low_cardinality_cache
clickhouse-gh bot added a commit that referenced this pull request Dec 12, 2025
@clickhouse-gh
Merge pull request #92021 from ClickHouse/backport/25.11/92009
14917cd 
Backport #92009 to 25.11: Fix flaky 00906_low_cardinality_cache
@fm4v fm4v self-assigned this Dec 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@fm4v fm4v

Labels

pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore pr-build Pull request with build/testing/packaging improvement pr-must-backport Pull request should be backported intentionally. Use this label with great care! pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Test 00906_low_cardinality_cache is flaky

3 participants

@fm4v @robot-ch-test-poll1 @robot-ch-test-poll4