---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36204/#review90727
---
Ship it!
LGTM! Could you please add a follow up patch to test this
On July 6, 2015, 5:50 p.m., Jie Yu wrote:
src/slave/slave.cpp, line 4379
https://reviews.apache.org/r/36204/diff/1/?file=1000223#file1000223line4379
Per my comments above, the logic here needs to be adjusted. We need to
apply checkpointed resource to info.resources() here.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36204/
---
(Updated July 7, 2015, 4:20 p.m.)
Review request for mesos, Ben Mahler, Jie
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36204/#review90510
---
Thanks Bartek! See my detailed comments.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36204/#review90516
---
Patch looks great!
Reviews applied: [36204]
All tests passed.
-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36204/
---
Review request for mesos, Ben Mahler, Jie Yu, Niklas Nielsen, Szymon Konefal,