Kevin Klues created MESOS-5178:
----------------------------------

             Summary: Add logic to validate for non-fractional GPU requests in 
the master
                 Key: MESOS-5178
                 URL: https://issues.apache.org/jira/browse/MESOS-5178
             Project: Mesos
          Issue Type: Task
            Reporter: Kevin Klues
            Assignee: Kevin Klues


We should not put this logic directly into the  'Resources::validate()' 
function.
The primary reason is that the existing 'Resources::validate()' function 
doesn't consider the semantics of any particular resource when performing its 
validation (it only makes sure that the fields in the 'Resource' protobuf 
message are correctly formed). Since a fractional 'gpus' resources is actually 
well-formed (and only semantically incorrect), we should push this validation 
logic up into the master.
    
Moreover, the existing logic to construct a 'Resources' object from a 
'RepeatedPtrField<Resource>' silently drops any resources that don't pass 
'Resources::validate()'. This means that if we were to push the non-fractional 
'gpus' validation into 'Resources::validate()', the 'gpus' resources would just 
be silently dropped rather than causing a TASK_ERROR in the master. This is 
obviously *not* the desired behaviour.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to