Temporarily disable server-side AST fuzzer in Stress Test and BuzzHouse by alexey-milovidov · Pull Request #100439 · ClickHouse/ClickHouse · GitHub
Skip to content

Temporarily disable server-side AST fuzzer in Stress Test and BuzzHouse#100439

Merged
qoega merged 1 commit into
masterfrom
disable-ast-fuzzer-stress-buzzhouse
Mar 23, 2026
Merged

Temporarily disable server-side AST fuzzer in Stress Test and BuzzHouse#100439
qoega merged 1 commit into
masterfrom
disable-ast-fuzzer-stress-buzzhouse

Conversation

@alexey-milovidov

@alexey-milovidov alexey-milovidov commented Mar 23, 2026

Copy link
Copy Markdown
Member

Temporarily disable ast_fuzzer_runs in Stress Test and BuzzHouse CI configs while stability issues are being addressed.

Alternative to full revert in #100437 — this only removes the CI config settings without reverting the C++ code improvements.

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):

...

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

Version info

  • Merged into: 26.4.1.152

Set `ast_fuzzer_runs` back to 0 (default) in stress test and BuzzHouse
CI configs while stability issues are being addressed.

See #100437

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

clickhouse-gh Bot commented Mar 23, 2026

Copy link
Copy Markdown
Contributor

@clickhouse-gh clickhouse-gh Bot added the pr-ci label Mar 23, 2026
Algunenano added a commit to Algunenano/ClickHouse that referenced this pull request Mar 23, 2026
The stress test profile sets `ast_fuzzer_runs=5`, enabling server-side
AST fuzzing for all queries. When the hung check runs its
`SELECT count() FROM system.processes` query via HTTP, the fuzzer
mutates it and spawns additional queries that appear in the processlist
— creating a self-referential loop where the hung check always sees
> 0 queries.

Two fixes:
1. Add `ast_fuzzer_runs=0` to the default HTTP params in
   `clickhouse_execute_http`, so infrastructure queries are not fuzzed.
2. When `get_processlist_with_stacktraces` times out, re-check
   `get_processlist_size` — if queries finished, don't fail.

References: ClickHouse#100439

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@qoega qoega added this pull request to the merge queue Mar 23, 2026
@qoega qoega self-assigned this Mar 23, 2026
Merged via the queue into master with commit 492097f Mar 23, 2026
152 checks passed
@qoega qoega deleted the disable-ast-fuzzer-stress-buzzhouse branch March 23, 2026 16:36
@robot-ch-test-poll4 robot-ch-test-poll4 added the pr-synced-to-cloud The PR is synced to the cloud repo label Mar 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

3 participants