-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/67274/
-----------------------------------------------------------
(Updated May 30, 2018, 11:50 p.m.)
Review request for mesos, Jie Yu, Joris Van Remoortere, Michael Park, and Neil
Conway.
Changes
-------
Do not reject explicitly zero resources. This rolls this patch back to revision
2.
Bugs: MESOS-8945
https://issues.apache.org/jira/browse/MESOS-8945
Repository: mesos
Description (updated)
-------
Internally values of scalar resources are stored in a fixed point
representation. This can lead to (expected) precision loss.
This patch updates the resource validation ensuring that values of
scalar resources are either zero or large enough so that they are
still represented as non-zero values after conversion to the internal
fixed point format. With that we can still parse resources containing
resource values explictly set to zero to be able to distinguish empty
and abset resources, but prevent propagation of values accidentially
rounded to an effectively empty result.
While this patch should not change valid, intended use cases, it might
change the behavior for erroneous, already broken framework workflows.
Diffs (updated)
-----
src/common/resources.cpp 192d086e60b81c0ae8e26d4120beee2034f63d3e
src/tests/master_tests.cpp 6efcbd12902de32428cb7f23ed0d3967a18f749d
src/tests/resources_tests.cpp 1201efec3ff0abc268f896c1fe2330571390b3fd
Diff: https://reviews.apache.org/r/67274/diff/5/
Changes: https://reviews.apache.org/r/67274/diff/4-5/
Testing
-------
`make check`
Thanks,
Benjamin Bannier
