Fix flaky 04075_analyzer_inline_view_subcolumns by vdimir · Pull Request #102022 · ClickHouse/ClickHouse · GitHub
Skip to content

Fix flaky 04075_analyzer_inline_view_subcolumns#102022

Merged
devcrafter merged 2 commits into
masterfrom
vdimir/04075_analyzer_inline_view_subcolumns_fix
Apr 9, 2026
Merged

Fix flaky 04075_analyzer_inline_view_subcolumns#102022
devcrafter merged 2 commits into
masterfrom
vdimir/04075_analyzer_inline_view_subcolumns_fix

Conversation

@vdimir

@vdimir vdimir commented Apr 8, 2026

Copy link
Copy Markdown
Member

Changelog category (leave one):

  • CI Fix or Improvement (changelog entry is not required)

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

...

Repro:

./tests/clickhouse-test -b ~/clickhouse 04075_analyzer_inline_view_subcolumns --merge-tree-settings "--map_serialization_version basic --map_serialization_version_for_zero_level_parts with_buckets --max_buckets_in_map 68 --map_buckets_strategy sqrt --map_buckets_min_avg_size 1"

Version info

  • Merged into: 26.4.1.720

@clickhouse-gh

clickhouse-gh Bot commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

@clickhouse-gh clickhouse-gh Bot added the pr-ci label Apr 8, 2026
@vdimir vdimir mentioned this pull request Apr 8, 2026
1 task
@alexey-milovidov

Copy link
Copy Markdown
Member

The MSan stress test failure (MemorySanitizer: use-of-uninitialized-value, STID 4179-5154 or 4148-3044) is a known pre-existing issue unrelated to this PR. Fix: #102158

alexey-milovidov added a commit that referenced this pull request Apr 9, 2026
The test failed in CI (Stateless tests, amd_asan_ubsan, distributed plan)
because the wait loop that polls `system.query_log` filtered by
`Settings['query_cache_tag']`, and this filter intermittently returned
no rows despite the entries existing. The exact cause is unclear but
may be related to how the large number of randomized settings interacts
with the `Settings` Map column access under ASan.

The `query_cache_tag` filter is redundant for isolation — the `query_id`
values already include the unique per-run database name. The tag is still
set in the HTTP URLs so that cache entries are properly isolated between
test runs; it is only removed from the `query_log` WHERE clauses.

CI report: https://s3.amazonaws.com/clickhouse-test-reports/json.html?PR=102022&sha=e0736f2b5221c7470a2381247890943ae675122e
PR: #102022

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@alexey-milovidov

Copy link
Copy Markdown
Member

The flaky failure of 02494_query_cache_http_introspection in this PR's CI is addressed by #102165.

1 similar comment
@alexey-milovidov

Copy link
Copy Markdown
Member

The flaky failure of 02494_query_cache_http_introspection in this PR's CI is addressed by #102165.

@devcrafter devcrafter self-assigned this Apr 9, 2026
@devcrafter

Copy link
Copy Markdown
Member

@devcrafter devcrafter added this pull request to the merge queue Apr 9, 2026
Merged via the queue into master with commit 4e3e2a5 Apr 9, 2026
160 of 164 checks passed
@devcrafter devcrafter deleted the vdimir/04075_analyzer_inline_view_subcolumns_fix branch April 9, 2026 16:22
@robot-clickhouse-ci-1 robot-clickhouse-ci-1 added the pr-synced-to-cloud The PR is synced to the cloud repo label Apr 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🍃 green ci 🌿 Fixing flaky tests in CI pr-ci 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.

4 participants