[CIP-19][CELEBORN-1937] QuotaManager should support app threshold#3651
[CIP-19][CELEBORN-1937] QuotaManager should support app threshold#3651leixm wants to merge 2 commits intoapache:mainfrom
Conversation
|
@AngersZhuuuu @RexXiong Can you help review? |
|
Overall LGTM, but I also have a couple of questions regarding the expiration strategy when multiple apps exceed the app-level quota:
by claude |
|
|
@leixm How's your plan with prioritized apps support? Will you support scheduling IO by app priorities? |
What changes were proposed in this pull request?
Reopen from #3336, QuotaManager supports app threshold.
Why are the changes needed?
For CIP-19, prioritize cleaning up abnormally large shuffle apps, then sort by app priority and consumption.
Does this PR resolve a correctness bug?
No.
Does this PR introduce any user-facing change?
Yes.
How was this patch tested?
UTs.