Added resource manager for tracking and planning resource usage withi…#376
Added resource manager for tracking and planning resource usage withi…#376Nitin-Kashyap wants to merge 1 commit intoopenlookeng:masterfrom
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: Nitin-Kashyap The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
/sync |
|
@sraghunandan: This pr has been synchronized to the Gitee Repository DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the opensourceways/test-infra repository. |
Nitin-Kashyap
force-pushed
the
feature-resourceManager
branch
2 times, most recently
from
74624f1 to
ad667b7
Compare
|
/sync |
|
@sraghunandan: This pr has been synchronized to the Gitee Repository DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the opensourceways/test-infra repository. |
Nitin-Kashyap
force-pushed
the
feature-resourceManager
branch
from
ad667b7 to
af914b3
Compare
|
/sync |
|
@sraghunandan: This pr has been synchronized to the Gitee Repository DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the opensourceways/test-infra repository. |
Nitin-Kashyap
force-pushed
the
feature-resourceManager
branch
2 times, most recently
from
fb39d5c to
0b76cf1
Compare
|
/sync |
|
@sraghunandan: This pr has been synchronized to the Gitee Repository DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the opensourceways/test-infra repository. |
…n and across queries
Nitin-Kashyap
force-pushed
the
feature-resourceManager
branch
from
0b76cf1 to
b820f84
Compare
|
/sync |
|
@sraghunandan: This pr has been synchronized to the Gitee Repository DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the opensourceways/test-infra repository. |
| long cachedMemoryUsage = queryStateMachine.getResourceGroupManager().getCachedMemoryUsage(queryStateMachine.getResourceGroup()); | ||
| long softReservedMemory = queryStateMachine.getResourceGroupManager().getSoftReservedMemory(queryStateMachine.getResourceGroup()); | ||
|
|
||
| /* Todo(Nitin K) replace with smart scheduler and pull the resource limit from QueryResourceManager/Service */ |
| @Override | ||
| public void setResourceLimit(DataSize memoryLimit, Duration cpuLimit, DataSize ioLimit) | ||
| { | ||
| /* Todo(Nitin K) check estimates correctness before updating limits! */ |
3 participants
…n and across queries
What type of PR is this?
/kind feature
What does this PR do / why do we need it:
adds a resource manager placeholder for monitoring and resource planning
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewers: