mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jiang Yan Xu <...@jxu.me>
Subject Re: Review Request 51867: Add a member variable for the agent's total resources.
Date Fri, 16 Sep 2016 18:23:46 GMT

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

(Updated Sept. 16, 2016, 11:23 a.m.)


Review request for mesos, Anindya Sinha and Neil Conway.


Changes
-------

Comments. NNFR.


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


Repository: mesos


Description
-------

`totalResources` is updated by CheckpointResourcesMessages and doesn't
have to be re-calculated in Slave::usage(). This change allows us to
expose `totalResources` for other purposes as well.


Diffs (updated)
-----

  src/slave/slave.hpp 13c76d1eaea2b49519948c9116e5db5caf9407ea 
  src/slave/slave.cpp 7f99e4610d06ebadbef48ce314fec6ad04acb307 

Diff: https://reviews.apache.org/r/51867/diff/


Testing
-------

make check.


Thanks,

Jiang Yan Xu


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