Skip to content

Fix flaky test 01111_create_drop_replicated_db_stress#97592

Merged
alexey-milovidov merged 1 commit intomasterfrom
fix-01111-create-drop-replicated-db-stress-flaky
Feb 22, 2026
Merged

Fix flaky test 01111_create_drop_replicated_db_stress#97592
alexey-milovidov merged 1 commit intomasterfrom
fix-01111-create-drop-replicated-db-stress-flaky

Conversation

@alexey-milovidov
Copy link
Copy Markdown
Member

Summary

  • Filter TIMEOUT_EXCEEDED exceptions in create_table and alter_table functions in the stress test
  • The DDLGuard has a hard-coded 1000 ms timeout for acquiring a shared database lock, which is insufficient when DROP DATABASE holds the exclusive lock longer under CI load
  • The sync_db function already filtered this exception; now create_table and alter_table do too

Fixes #97539

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)

🤖 Generated with Claude Code

The `DDLGuard` constructor has a hard-coded 1000 ms timeout for acquiring
a shared database lock. When `DROP DATABASE` holds the exclusive lock
longer than 1 second (e.g. due to ZooKeeper cleanup on a loaded CI
machine), table-level DDL operations (`CREATE TABLE`, `ALTER TABLE`) fail
with `TIMEOUT_EXCEEDED`.

The `sync_db` function already filtered this exception, but `create_table`
and `alter_table` did not. Add `TIMEOUT_EXCEEDED` filtering to both, since
this timeout is expected under concurrent create/drop stress.

#97539

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Feb 22, 2026

Workflow [PR], commit [e738b97]

Summary:

@clickhouse-gh clickhouse-gh bot added the pr-ci label Feb 22, 2026
@alexey-milovidov alexey-milovidov self-assigned this Feb 22, 2026
@alexey-milovidov alexey-milovidov added this pull request to the merge queue Feb 22, 2026
Merged via the queue into master with commit a803243 Feb 22, 2026
148 checks passed
@alexey-milovidov alexey-milovidov deleted the fix-01111-create-drop-replicated-db-stress-flaky branch February 22, 2026 15:23
@robot-ch-test-poll4 robot-ch-test-poll4 added the pr-synced-to-cloud The PR is synced to the cloud repo label Feb 22, 2026
Algunenano pushed a commit to Algunenano/ClickHouse that referenced this pull request Feb 24, 2026
…drop-replicated-db-stress-flaky

Fix flaky test 01111_create_drop_replicated_db_stress
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.

Flaky test: 01111_create_drop_replicated_db_stress

2 participants