Skip to content

Fix bad test 03238_create_or_replace_view_atomically_with_replicated_engine.sh#93664

Merged
alexey-milovidov merged 4 commits intomasterfrom
fix-bad-test-5
Jan 9, 2026
Merged

Fix bad test 03238_create_or_replace_view_atomically_with_replicated_engine.sh#93664
alexey-milovidov merged 4 commits intomasterfrom
fix-bad-test-5

Conversation

@alexey-milovidov alexey-milovidov changed the title Fix bad test `` Fix bad test 03238_create_or_replace_view_atomically_with_replicated_engine.sh Jan 8, 2026
@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Jan 8, 2026

Workflow [PR], commit [de5a9fa]

Summary:

@clickhouse-gh clickhouse-gh bot added the pr-ci label Jan 8, 2026
@Algunenano
Copy link
Copy Markdown
Member

cc @tuanpach I only appear as author because I reverted the original PR because of the bad tests and open a revert of revert so it was fixed (and it wasn't)

@alexey-milovidov
Copy link
Copy Markdown
Member Author

Ok. It looks like OOM due to too many concurrent invocations of clickhouse-client, while under sanitizers, it takes a lot of RAM.

@tuanpach
Copy link
Copy Markdown
Member

tuanpach commented Jan 9, 2026

Thanks for tidying the test. export-f and bash are unnecessary.

@tuanpach tuanpach self-assigned this Jan 9, 2026
@alexey-milovidov alexey-milovidov added this pull request to the merge queue Jan 9, 2026
Merged via the queue into master with commit 86f2b5b Jan 9, 2026
132 checks passed
@alexey-milovidov alexey-milovidov deleted the fix-bad-test-5 branch January 9, 2026 04:21
@robot-ch-test-poll2 robot-ch-test-poll2 added the pr-synced-to-cloud The PR is synced to the cloud repo label Jan 9, 2026
@thevar1able thevar1able added the post-approved Approved, but after the PR is merged. label Mar 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

post-approved Approved, but after the PR is merged. 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: 03238_create_or_replace_view_atomically_with_replicated_engine

5 participants