-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12341/
-----------------------------------------------------------
(Updated July 18, 2013, 12:25 a.m.)
Review request for mesos and Benjamin Hindman.
Changes
-------
Moved the role in the resource input/output format to be next to the resource
name:
"cpus(role1):1;..."
Bugs: MESOS-505
https://issues.apache.org/jira/browse/MESOS-505
Repository: mesos
Description
-------
Adds parsing of roles to the Resources class. Also updates parse to return a
Try, in the case that parsing fails, and adds tests for role parsing and for
Resources operators in the presence of roles.
Diffs (updated)
-----
src/common/resources.hpp 42dfb6a
src/common/resources.cpp 8bc1057
src/master/drf_sorter.cpp 510acc3
src/slave/slave.cpp 9a2c671
src/tests/gc_tests.cpp 1745800
src/tests/master_tests.cpp 5ac4d5f
src/tests/resource_offers_tests.cpp beab2d8
src/tests/resources_tests.cpp 67a11b3
src/tests/slave_recovery_tests.cpp b0843d7
src/tests/sorter_tests.cpp 619e786
Diff: https://reviews.apache.org/r/12341/diff/
Testing
-------
make check
Thanks,
Thomas Marshall
