Skip to content

Replace SchedulePool with a separate thread#89040

Merged
divanik merged 2 commits intomasterfrom
divanik/replace_scheduler_pool_with_thread
Oct 28, 2025
Merged

Replace SchedulePool with a separate thread#89040
divanik merged 2 commits intomasterfrom
divanik/replace_scheduler_pool_with_thread

Conversation

@divanik
Copy link
Member

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

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

@clickhouse-gh
Copy link
Contributor

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

Workflow [PR], commit [53e86aa]

Summary: ❌

job_name test_name status info comment
Stateless tests (amd_ubsan, parallel) failure
00900_long_parquet_load FAIL cidb, flaky
Docker server image failure
docker library image test failure 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
@divanik divanik requested a review from alesapin October 27, 2025 13:02
@divanik divanik marked this pull request as ready for review October 27, 2025 13:02
@divanik divanik changed the title [[DRAFT]] Replace SchedulePool with a separate thread Replace SchedulePool with a separate thread Oct 27, 2025
@divanik divanik enabled auto-merge October 27, 2025 13:17
@alesapin alesapin self-assigned this Oct 28, 2025
@divanik
Copy link
Member Author

divanik commented Oct 28, 2025

Issue with docker seems to be fixed here
00900_long_parquet_load

@divanik divanik added this pull request to the merge queue Oct 28, 2025
@divanik divanik added the pr-must-backport Pull request should be backported intentionally. Use this label with great care! label Oct 28, 2025
Merged via the queue into master with commit 4d1bab1 Oct 28, 2025
238 of 244 checks passed
@divanik divanik deleted the divanik/replace_scheduler_pool_with_thread branch October 28, 2025 11:42
@robot-ch-test-poll3 robot-ch-test-poll3 added the pr-synced-to-cloud The PR is synced to the cloud repo label Oct 28, 2025
@robot-ch-test-poll1 robot-ch-test-poll1 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 28, 2025
This was referenced Oct 29, 2025
Closed
Closed
Merged
robot-clickhouse added a commit that referenced this pull request Oct 29, 2025
@robot-clickhouse
Merge pull request #89146 from ClickHouse/cherrypick/25.8/89040
ea4680b 
Cherry pick #89040 to 25.8: Replace SchedulePool with a separate thread
robot-clickhouse added a commit that referenced this pull request Oct 29, 2025
@robot-clickhouse
Backport #89040 to 25.8: Replace SchedulePool with a separate thread
a5d5db8
This was referenced Oct 29, 2025
Merged
Merged
robot-clickhouse added a commit that referenced this pull request Oct 29, 2025
@robot-clickhouse
Merge pull request #89148 from ClickHouse/cherrypick/25.9/89040
4357627 
Cherry pick #89040 to 25.9: Replace SchedulePool with a separate thread
robot-clickhouse added a commit that referenced this pull request Oct 29, 2025
@robot-clickhouse
Backport #89040 to 25.9: Replace SchedulePool with a separate thread
99756e5
This was referenced Oct 29, 2025
Closed
Merged
robot-clickhouse added a commit that referenced this pull request Oct 29, 2025
@robot-clickhouse
Merge pull request #89150 from ClickHouse/cherrypick/25.10/89040
c752609 
Cherry pick #89040 to 25.10: Replace SchedulePool with a separate thread
robot-clickhouse added a commit that referenced this pull request Oct 29, 2025
@robot-clickhouse
Backport #89040 to 25.10: Replace SchedulePool with a separate thread
c7f424a
@robot-clickhouse robot-clickhouse mentioned this pull request Oct 29, 2025
Merged
@robot-ch-test-poll4 robot-ch-test-poll4 added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Oct 29, 2025
divanik added a commit that referenced this pull request Oct 29, 2025
@divanik
Merge pull request #89151 from ClickHouse/backport/25.10/89040
87ca7bd 
Backport #89040 to 25.10: Replace SchedulePool with a separate thread
divanik added a commit that referenced this pull request Oct 29, 2025
@divanik
Merge pull request #89147 from ClickHouse/backport/25.8/89040
132a0af 
Backport #89040 to 25.8: Replace SchedulePool with a separate thread
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@scanhex12 scanhex12 scanhex12 approved these changes

@alesapin alesapin Awaiting requested review from alesapin

Assignees

@alesapin alesapin

Labels

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 Pull request should be backported intentionally. Use this label with great care! 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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@divanik @scanhex12 @alesapin @robot-ch-test-poll1 @robot-ch-test-poll4 @robot-ch-test-poll3