Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change cluster resource to use DispatchManager #899

Merged
merged 1 commit into from Jun 4, 2019

Conversation

2 participants
@dain
Copy link
Member

commented Jun 3, 2019

QueryManager does not manage queued queries, and DispatchManager has all stats necessary for cluster resource.

Fixes #894

@cla-bot cla-bot bot added the cla-signed label Jun 3, 2019

@martint

martint approved these changes Jun 4, 2019

@martint

martint approved these changes Jun 4, 2019

Change cluster resource to use DispatchManager
QueryManager does not manage queued queries, and DispatchManager has all stats necessary for cluster resource.

@dain dain force-pushed the dain:fix-ui-queued-stat branch from 6e4824e to 86fa853 Jun 4, 2019

@dain dain merged commit 86fa853 into prestosql:master Jun 4, 2019

1 of 3 checks passed

Travis CI - Branch Build Errored
Details
Travis CI - Pull Request Build Created
Details
verification/cla-signed
Details

@dain dain deleted the dain:fix-ui-queued-stat branch Jun 4, 2019

@dain dain added this to the 314 milestone Jun 4, 2019

@dain dain referenced this pull request Jun 4, 2019

Closed

Release notes for 314 #879

2 of 6 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.