You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@mesos.apache.org by Jay Guo <gu...@cn.ibm.com> on 2016/04/06 05:28:04 UTC

Review Request 45794: Change RateLimiter::acquire() to be const.

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

Review request for mesos.


Repository: mesos


Description
-------

In some cases, RateLimiter is wrapped in Shared<> which enforces
const access of object. We want to avoid const_cast.


Diffs
-----

  3rdparty/libprocess/include/process/limiter.hpp 25254cdc2cb095b96d58d3f73dafc0ea48824998 

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


Testing
-------


Thanks,

Jay Guo

Re: Review Request 45794: Change RateLimiter::acquire() to be const.

Posted by Jay Guo <gu...@cn.ibm.com>.

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

(Updated April 6, 2016, 3:35 a.m.)


Review request for mesos.


Repository: mesos


Description
-------

In some cases, RateLimiter is wrapped in Shared<> which enforces
const access of object. We want to avoid const_cast.


Diffs
-----

  3rdparty/libprocess/include/process/limiter.hpp 25254cdc2cb095b96d58d3f73dafc0ea48824998 

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


Testing (updated)
-------

make check


Thanks,

Jay Guo