Skip to content

Fix flaky 00365_statistics_in_formats with parallel replicas#96970

Merged
alexey-milovidov merged 1 commit intomasterfrom
fix-00365-flaky-parallel-replicas
Feb 15, 2026
Merged

Fix flaky 00365_statistics_in_formats with parallel replicas#96970
alexey-milovidov merged 1 commit intomasterfrom
fix-00365-flaky-parallel-replicas

Conversation

@alexey-milovidov
Copy link
Copy Markdown
Member

The test checks rows_read statistics in JSON/JSONCompact/XML output formats. With parallel replicas, there is a race condition where the output format's finalizeImpl writes the statistics footer before all progress from connection draining has been reported, resulting in rows_read being 0 instead of 10.

Add the no-parallel-replicas tag to avoid the flakiness.

Closes #85785

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)

The test checks `rows_read` statistics in JSON/JSONCompact/XML output
formats. With parallel replicas, there is a race condition where the
output format's `finalizeImpl` writes the statistics footer before all
progress from connection draining has been reported, resulting in
`rows_read` being 0 instead of 10.

Closes #85785

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@alexey-milovidov
Copy link
Copy Markdown
Member Author

We should investigate the root cause: why format statistics are not received correctly with parallel replicas? Can we fix that?

@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Feb 15, 2026

Workflow [PR], commit [7341a68]

Summary:

@clickhouse-gh clickhouse-gh bot added the pr-ci label Feb 15, 2026
@alexey-milovidov
Copy link
Copy Markdown
Member Author

This is a suppression of a real, but unimportant problem. The root cause fix is here: #96978

@alexey-milovidov alexey-milovidov merged commit 8e89dae into master Feb 15, 2026
133 of 134 checks passed
@alexey-milovidov alexey-milovidov deleted the fix-00365-flaky-parallel-replicas branch February 15, 2026 22:11
@alexey-milovidov alexey-milovidov self-assigned this Feb 15, 2026
@robot-clickhouse-ci-1 robot-clickhouse-ci-1 added the pr-synced-to-cloud The PR is synced to the cloud repo label Feb 15, 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.

00365_statistics_in_formats is flaky with parallel replicas

2 participants