-
Notifications
You must be signed in to change notification settings - Fork 327
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
scheduler: ElasticQuota runtime is no longer calculated when not needed #1855
scheduler: ElasticQuota runtime is no longer calculated when not needed #1855
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #1855 +/- ##
==========================================
+ Coverage 67.07% 67.09% +0.01%
==========================================
Files 407 407
Lines 45705 45702 -3
==========================================
+ Hits 30658 30663 +5
+ Misses 12820 12812 -8
Partials 2227 2227
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Signed-off-by: Joseph <joseph.t.lee@outlook.com>
2a9f88d
to
143c18b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wait
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: buptcozy The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
…ed (koordinator-sh#1855) Signed-off-by: Joseph <joseph.t.lee@outlook.com>
…ed (#1855) Signed-off-by: Joseph <joseph.t.lee@outlook.com>
…ed (koordinator-sh#1855) Signed-off-by: Joseph <joseph.t.lee@outlook.com>
…ed (koordinator-sh#1855) Signed-off-by: Joseph <joseph.t.lee@outlook.com>
…ed (#1855) Signed-off-by: Joseph <joseph.t.lee@outlook.com>
…ed (koordinator-sh#1855) Signed-off-by: Joseph <joseph.t.lee@outlook.com>
Ⅰ. Describe what this PR does
Changes have been made in PR #1839. When
runtime
is not needed,max
will be used as the upper limit of ElasticQuota. But it hasn't been done thoroughly yet. Since there is no need to useruntime
, there is no need to calculate runtime in the PreFilter stage. In addition,GetQuotaSummary
is an externally exposed interface. It only needs to read the most recentruntime
, and there is no need to recalculateruntime
. If we need to computeruntime
more aggressively in the future, we should consider implementing a more efficient, cohesive method to support it.Ⅱ. Does this pull request fix one issue?
Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist
make test