Skip to content

Fix Disaggregated Coordinator Weighted Scheduling#18595

Merged
tdcmeehan merged 1 commit intoprestodb:masterfrom
swapsmagic:disagg_coordinator_weighted_scheduling_fix
Nov 2, 2022
Merged

Fix Disaggregated Coordinator Weighted Scheduling#18595
tdcmeehan merged 1 commit intoprestodb:masterfrom
swapsmagic:disagg_coordinator_weighted_scheduling_fix

Conversation

@swapsmagic
Copy link
Contributor

Resource group manager was considering locally running queries count to decide on the group weight and prioritze next running query or eligible group. With the fix, it considers all queries running in the cluster for a given resource group to decide the scheduling weight.

Test plan - unit test

== RELEASE NOTES ==

General Changes
* Fix weighted scheduling for disaggregated coordinator

@swapsmagic swapsmagic requested a review from a team as a code owner October 31, 2022 23:44
@swapsmagic swapsmagic requested a review from presto-oss October 31, 2022 23:44
@swapsmagic swapsmagic changed the title Fix Disaggregated Coordinator Weighted Scheduling Fix Fix Disaggregated Coordinator Weighted Scheduling Oct 31, 2022
@swapsmagic swapsmagic force-pushed the disagg_coordinator_weighted_scheduling_fix branch 2 times, most recently from 0eb2c13 to 3261013 Compare November 1, 2022 16:47
Resource group manager was considering locally running queries count to decide on the group weight
and prioritze next running query or eligible group. With the fix, it considers all queries running in
the cluster for a given resource group to decide the scheduling weight.
@swapsmagic swapsmagic force-pushed the disagg_coordinator_weighted_scheduling_fix branch from 3261013 to 5b7fde3 Compare November 1, 2022 17:48
@swapsmagic swapsmagic requested a review from a team November 2, 2022 22:15
@tdcmeehan tdcmeehan merged commit 208b60e into prestodb:master Nov 2, 2022
@wanglinsong wanglinsong mentioned this pull request Jan 12, 2023
30 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants