craigcondit commented on code in PR #143:
URL:
https://github.com/apache/yunikorn-scheduler-interface/pull/143#discussion_r1772272587
##########
scheduler-interface-spec.md:
##########
@@ -607,12 +607,10 @@ message NodeInfo {
// node or schedule more allocations accordingly.
Resource schedulableResource = 4;
- // when the scheduler is co-exist with some other schedulers, some node
- // resources might be occupied (allocated) by other schedulers.
- Resource occupiedResource = 5;
-
reserved 6;
reserved "existingAllocations";
+ reserved 5;
Review Comment:
I think we should split this PR into two phases -- one which adds the new
constants, and a second which removes the old API (to be committed after all
changes have been made in both core and shim). Tracking non-YK pods is a huge
feature, and this gives us more flexibility in the interim.
--
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]
