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

Posted to dev@mesos.apache.org by Benjamin Hindman <be...@berkeley.edu> on 2015/03/02 07:03:17 UTC

Review Request 31610: Simplified 'cleanupCache' by adjusting cache sizes earlier.

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

Review request for mesos and Bernd Mathiske.


Repository: mesos


Description
-------

Rather than waiting to asynchronously adjust the claimed cache space in 'cleanupCache' which seemed to be a very important part of the "success" flow of the algorithm this was moved earlier (right to where success occurs, in FetcherProcess::___fetch). This enabled making FetcherProcess::cleanupCache be more focused on cleaning up new cache entries when mesos-fetcher fails.


Diffs
-----

  src/slave/containerizer/fetcher.hpp 1db0eaf002c8d0eaf4e0391858e61e0912b35829 
  src/slave/containerizer/fetcher.cpp 9e9e9d0eb6b0801d53dec3baea32a4cd4acdd5e2 

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


Testing
-------

make check


Thanks,

Benjamin Hindman