Github user ArtRand commented on the issue:

    https://github.com/apache/spark/pull/19390
  
    Just to reiterate our conversation here. 
    
    The `AllocationInfo` tells the scheduler which `role`’s allocation of 
resources the `Offer` is coming from, whereas the `ReservationInfo` tells the 
scheduler whether or not the `Offer` contains reserved resources. We needed the 
latter to effectively user dynamically reserved resources.
    
    From your (old) PR it looks like the `AllocationInfo` is mostly just 
“forwarded”, i.e. there isn’t any logic around it’s contents. So your 
new PR removes references to `AllocationInfo` to prevent breaking with Mesos 
1.3- and remain non-MULTI_ROLE.
    
    The purpose of `AllocationInfo` is (in the future) to be able to have some 
logic around which tasks get launched on resources allocated to various roles 
that a framework is subscribed as - but we don’t have that yet and isn't 
addressed in this patch. 


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to