Skip to content

Disable the new parquet reader in the CI#89067

Merged
nikitamikhaylov merged 1 commit intomasterfrom
disable-parquet
Oct 27, 2025
Merged

Disable the new parquet reader in the CI#89067
nikitamikhaylov merged 1 commit intomasterfrom
disable-parquet

Conversation

@nikitamikhaylov
Copy link
Copy Markdown
Member

@nikitamikhaylov nikitamikhaylov commented Oct 27, 2025 •
edited by clickhouse-gh bot
Loading

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

Details

The test is extremely flaky. Resolves #88893

@nikitamikhaylov nikitamikhaylov added the 🍃 green ci 🌿 Fixing flaky tests in CI label Oct 27, 2025
@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Oct 27, 2025 •
edited
Loading

Workflow [PR], commit [89d9123]

Summary: ⏳

job_name test_name status info comment
Stress test (amd_debug) failure
Server died FAIL cidb
Hung check failed, possible deadlock found (see hung_check.log) FAIL cidb
Killed by signal (in clickhouse-server.log) FAIL cidb
Fatal message in clickhouse-server.log (see fatal_messages.txt) FAIL cidb
Killed by signal (output files) FAIL cidb
Found signal in gdb.log FAIL cidb

@clickhouse-gh clickhouse-gh bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label Oct 27, 2025
@pamarcos pamarcos self-assigned this Oct 27, 2025
pamarcos
pamarcos approved these changes Oct 27, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@pamarcos pamarcos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we add a comment or comment it out so that we can easily re-enable it once it's more stable?

@nikitamikhaylov
Copy link
Copy Markdown
Member Author

nikitamikhaylov commented Oct 27, 2025

I hope @al13n321 will create a "revert revert" PR almost immediately...

@nikitamikhaylov
Copy link
Copy Markdown
Member Author

nikitamikhaylov commented Oct 27, 2025

@Algunenano This PR #87600 is not in 25.10.

@Algunenano
Copy link
Copy Markdown
Member

Algunenano commented Oct 27, 2025

@Algunenano This PR #87600 is not in 25.10.

If I go to the commit in 58971a6 I see it's in v25.11.1.1-new, which means it's in the 25.10 branch. You can see it in the branch itself too https://github.com/ClickHouse/ClickHouse/commits/25.10/

@nikitamikhaylov nikitamikhaylov added this pull request to the merge queue Oct 27, 2025
Merged via the queue into master with commit 05d051f Oct 27, 2025
91 of 123 checks passed
@nikitamikhaylov nikitamikhaylov deleted the disable-parquet branch October 27, 2025 16:14
@robot-ch-test-poll robot-ch-test-poll added the pr-synced-to-cloud The PR is synced to the cloud repo label Oct 27, 2025
@robot-ch-test-poll4 robot-ch-test-poll4 added pr-backports-created-cloud deprecated label, NOOP pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR labels Oct 27, 2025
@robot-ch-test-poll1 robot-ch-test-poll1 mentioned this pull request Oct 27, 2025
Merged
robot-ch-test-poll1 added a commit that referenced this pull request Oct 27, 2025
@robot-ch-test-poll1
Merge pull request #89077 from ClickHouse/cherrypick/25.10/89067
3e543b7 
Cherry pick #89067 to 25.10: Disable the new parquet reader in the CI
robot-clickhouse added a commit that referenced this pull request Oct 27, 2025
@robot-clickhouse
Backport #89067 to 25.10: Disable the new parquet reader in the CI
a595c98
@robot-ch-test-poll1 robot-ch-test-poll1 mentioned this pull request Oct 27, 2025
Merged
@robot-clickhouse-ci-1 robot-clickhouse-ci-1 added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Oct 27, 2025
clickhouse-gh bot added a commit that referenced this pull request Oct 27, 2025
@clickhouse-gh
Merge pull request #89078 from ClickHouse/backport/25.10/89067
cda1143 
Backport #89067 to 25.10: Disable the new parquet reader in the CI
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pamarcos pamarcos pamarcos approved these changes

Assignees

@pamarcos pamarcos

Labels

🍃 green ci 🌿 Fixing flaky tests in CI pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore pr-backports-created-cloud deprecated label, NOOP pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR pr-not-for-changelog This PR should not be mentioned in the changelog pr-synced-to-cloud The PR is synced to the cloud repo v25.10-must-backport

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

00900_long_parquet_load is flaky

6 participants

@nikitamikhaylov @Algunenano @pamarcos @robot-ch-test-poll @robot-ch-test-poll4 @robot-clickhouse-ci-1