----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51879/ -----------------------------------------------------------
(Updated Dec. 18, 2016, 8:38 a.m.) Review request for mesos and Jiang Yan Xu. Changes ------- Addressed review comments. Bugs: MESOS-6062 https://issues.apache.org/jira/browse/MESOS-6062 Repository: mesos Description (updated) ------- When static resources indicate resources with a positive size, we use that for the resources on the agent. However, --resources can include resources with no size, which indicates that mesos agent determine the size of those resources from the agent and uses that information. Note that auto-detection of resources is allowed for all known resource types represented in JSON format only. Auto-detection is not done when the resources are represented in text format. With this change, JSON representation for disk resources that do not specify any value would not result in an error, but those resources will not be accounted for until a valid size is determined for such resources. A scalar value of -1 still results in an invalid resource. Diffs (updated) ----- include/mesos/resources.hpp c2e6d6680d9687f0bbfb73e84ffb1457072c4412 include/mesos/v1/resources.hpp 98f025b68c818c384a1174ef9b969f5c0776a7f1 src/common/resources.cpp 7dbeefc8fc159f28dc1542ad6a761213c65ff226 src/slave/containerizer/containerizer.cpp d46882baa904fd439bffb23c324828b777228f1c src/slave/containerizer/mesos/isolators/gpu/allocator.cpp 2e722691475c84afae14009014ea70cc0fdd0e65 src/v1/resources.cpp 859f7388035b855bf0cb60d5cbe746e6871bf833 Diff: https://reviews.apache.org/r/51879/diff/ Testing ------- Tests passed. Thanks, Anindya Sinha