Skip to content

Fix 03254_timeseries_to_grid_aggregate_function with PR#82998

Merged
alexey-milovidov merged 1 commit intomasterfrom
fix-03254_timeseries_to_grid_aggregate_function
Jul 1, 2025
Merged

Fix 03254_timeseries_to_grid_aggregate_function with PR#82998
alexey-milovidov merged 1 commit intomasterfrom
fix-03254_timeseries_to_grid_aggregate_function

Conversation

@devcrafter
Copy link
Copy Markdown
Member

@devcrafter devcrafter commented Jul 1, 2025

https://s3.amazonaws.com/clickhouse-test-reports/json.html?REF=master&sha=25f1e120baed0dda00234fa93499101491c08bf5&name_0=MasterCI&name_1=Stateless%20tests%20%28amd_binary%2C%20ParallelReplicas%2C%20s3%20storage%29

Become flaky after #80425 since parallel distributed insert select with PR can produce more parts with simple select

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Jul 1, 2025

Workflow [PR], commit [6cc72cc]

Summary:

job_name test_name status info comment
Stateless tests (amd_binary, ParallelReplicas, s3 storage) failure
Exception in test runner FAIL

@clickhouse-gh clickhouse-gh bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label Jul 1, 2025
@davenger davenger self-assigned this Jul 1, 2025
@alexey-milovidov
Copy link
Copy Markdown
Member

@devcrafter, thanks for the fix! But you forgot to review the test failures.
You need to notify @alesapin about the incident in our CI.

@devcrafter
Copy link
Copy Markdown
Member Author

@devcrafter, thanks for the fix! But you forgot to review the test failures. You need to notify @alesapin about the incident in our CI.

It's hard to forget if I don't know about it :) I just press Auto-merge for simple changes or for changes which consider good enough for merge. The rest is the mergeable/private sync jobs job

@alexey-milovidov alexey-milovidov disabled auto-merge July 1, 2025 23:09
@alexey-milovidov alexey-milovidov merged commit e06a04a into master Jul 1, 2025
118 of 122 checks passed
@alexey-milovidov alexey-milovidov deleted the fix-03254_timeseries_to_grid_aggregate_function branch July 1, 2025 23:09
@robot-ch-test-poll robot-ch-test-poll added the pr-synced-to-cloud The PR is synced to the cloud repo label Jul 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants