Skip to content

Fix std::out_of_range in ApplyWithSubqueryVisitor with dictGet#91022

Merged
azat merged 2 commits intoClickHouse:masterfrom
Avogar:fix-dict-get-cte
Nov 28, 2025
Merged

Fix std::out_of_range in ApplyWithSubqueryVisitor with dictGet#91022
azat merged 2 commits intoClickHouse:masterfrom
Avogar:fix-dict-get-cte

Conversation

@Avogar
Copy link
Copy Markdown
Member

@Avogar Avogar commented Nov 27, 2025

Changelog category (leave one):

  • Critical Bug Fix (crash, data loss, RBAC) or LOGICAL_ERROR

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

Fix std::out_of_range in CTE with dictGet with no arguments. Closes #91027

Fixes: #90860

@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Nov 27, 2025

Workflow [PR], commit [d87e5aa]

Summary:

job_name test_name status info comment
Stateless tests (arm_asan, targeted) failure
02015_async_inserts_4 FAIL cidb
02015_async_inserts_4 FAIL cidb
Stateless tests (amd_binary, ParallelReplicas, s3 storage, parallel) failure
03514_grace_hash_join_logical_error FAIL cidb, flaky
Stateless tests (amd_debug, parallel) failure
03702_alter_column_update_and_delete_with_secondary_index_drop FAIL cidb
Stateless tests (amd_tsan, parallel) failure
03212_variant_dynamic_cast_or_default FAIL cidb, flaky
Integration tests (amd_asan, db disk, old analyzer, 5/6) failure
test_dictionaries_update_and_reload/test.py::test_reload_while_loading FAIL cidb
Stress test (amd_debug) failure
Server died FAIL cidb
Hung check failed, possible deadlock found (see hung_check.log) FAIL cidb
Killed by signal (in clickhouse-server.log) FAIL cidb
Fatal message in clickhouse-server.log (see fatal_messages.txt) FAIL cidb
Killed by signal (output files) FAIL cidb
Found signal in gdb.log FAIL cidb
BuzzHouse (amd_debug) failure
Logical error: 'Inconsistent AST formatting: the query: FAIL cidb, flaky
BuzzHouse (amd_msan) failure
Unknown error FAIL cidb
OOM in dmesg FAIL cidb

@clickhouse-gh clickhouse-gh bot added pr-critical-bugfix pr-must-backport Pull request should be backported intentionally. Use this label with great care! labels Nov 27, 2025
@azat azat self-assigned this Nov 27, 2025
@azat azat enabled auto-merge November 27, 2025 21:25
@azat azat added this pull request to the merge queue Nov 28, 2025
Merged via the queue into ClickHouse:master with commit 59b3570 Nov 28, 2025
121 of 130 checks passed
@robot-clickhouse-ci-2 robot-clickhouse-ci-2 added the pr-synced-to-cloud The PR is synced to the cloud repo label Nov 28, 2025
@robot-clickhouse-ci-1 robot-clickhouse-ci-1 added the pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR label Nov 28, 2025
robot-ch-test-poll1 added a commit that referenced this pull request Nov 28, 2025
Cherry pick #91022 to 25.8: Fix std::out_of_range in ApplyWithSubqueryVisitor with dictGet
robot-clickhouse added a commit that referenced this pull request Nov 28, 2025
robot-ch-test-poll1 added a commit that referenced this pull request Nov 28, 2025
Cherry pick #91022 to 25.10: Fix std::out_of_range in ApplyWithSubqueryVisitor with dictGet
robot-clickhouse added a commit that referenced this pull request Nov 28, 2025
robot-ch-test-poll1 added a commit that referenced this pull request Nov 28, 2025
Cherry pick #91022 to 25.11: Fix std::out_of_range in ApplyWithSubqueryVisitor with dictGet
robot-clickhouse added a commit that referenced this pull request Nov 28, 2025
@robot-clickhouse robot-clickhouse added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Nov 28, 2025
azat added a commit that referenced this pull request Nov 28, 2025
Backport #91022 to 25.8: Fix std::out_of_range in ApplyWithSubqueryVisitor with dictGet
azat added a commit that referenced this pull request Nov 28, 2025
Backport #91022 to 25.10: Fix std::out_of_range in ApplyWithSubqueryVisitor with dictGet
azat added a commit that referenced this pull request Nov 28, 2025
Backport #91022 to 25.11: Fix std::out_of_range in ApplyWithSubqueryVisitor with dictGet
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore pr-critical-bugfix 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

Development

Successfully merging this pull request may close these issues.

Logical error: 'std::exception. Code: 1001, type: std::out_of_range, e.what() = InlinedVector::at(size_type) failed bounds check

5 participants