leixm commented on PR #3651: URL: https://github.com/apache/celeborn/pull/3651#issuecomment-4211941029
> Overall LGTM， but I also have a couple of questions regarding the expiration strategy when multiple apps exceed the app-level quota: > > 1. **Priority-based expiration:** When a user exceeds their overall quota and multiple apps also exceed the app-level threshold, could we prioritize which apps to expire based on app priority (or consumption)? This would allow more important apps to continue running while less critical ones are throttled first. > 2. **Incremental expiration:** After expiring a single app, if the remaining apps no longer exceed the quota, is it possible to avoid expiring them? This would minimize the impact to the user's workload by only expiring the minimum number of apps necessary to bring usage back under the quota limit. > > by claude 1. Subsequent PRs will be prioritized based on priority and consumption. 2. Incremental expiration was already supported before this PR. This PR aims to introduce a new app threshold based on fairness considerations. We don't want quotas to be consumed by a few apps. When the user/tenant/cluster quota is exceeded, apps exceeding the app threshold will be killed first. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
