mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Bannier <benjamin.bann...@mesosphere.io>
Subject Re: Review Request 67274: Rejected non-zero scalar resource values which be represented as zero.
Date Mon, 28 May 2018 15:11:54 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/67274/
-----------------------------------------------------------

(Updated May 28, 2018, 5:11 p.m.)


Review request for mesos, Jie Yu, Joris Van Remoortere, Michael Park, and Neil Conway.


Changes
-------

Allowed empty resources.


Bugs: MESOS-8945
    https://issues.apache.org/jira/browse/MESOS-8945


Repository: mesos


Description
-------

Internally values of scalar resources are stored in a fixed point
representation. This can lead to (expected) precision loss.

This patch adds 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.

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/resources_tests.cpp 1201efec3ff0abc268f896c1fe2330571390b3fd 


Diff: https://reviews.apache.org/r/67274/diff/3/

Changes: https://reviews.apache.org/r/67274/diff/2-3/


Testing
-------

`make check`


Thanks,

Benjamin Bannier


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message