---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/33159/#review86783
---
Patch looks great!
Reviews applied: [32961, 33159]
All tests
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/33159/#review86781
---
Ship it!
src/master/allocator/mesos/hierarchical.hpp
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/33159/
---
(Updated June 5, 2015, 8:53 a.m.)
Review request for mesos and Benjamin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/33159/
---
(Updated June 5, 2015, 1:35 p.m.)
Review request for mesos and Benjamin
On June 5, 2015, 5:42 p.m., Vinod Kone wrote:
src/master/allocator/mesos/hierarchical.hpp, lines 455-456
https://reviews.apache.org/r/33159/diff/4/?file=979945#file979945line455
I don't follow. Why are these CHECKs? Is there currently code in the
master that guarantees that these
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/33159/#review86818
---
src/master/allocator/mesos/hierarchical.hpp
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/33159/
---
(Updated May 29, 2015, 10:41 p.m.)
Review request for mesos and Benjamin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/33159/#review84882
---
Ship it!
src/master/master.cpp
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/33159/#review84893
---
src/master/allocator/mesos/hierarchical.hpp