Re: Review Request 36314: Added test for passing total slave's resources in ResourceUsage.

2015-07-14 Thread Bartek Plotka
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36314/ --- (Updated July 14, 2015, 11:09 a.m.) Review request for mesos, Jie Yu, Niklas

Re: Review Request 36314: Added test for passing total slave's resources in ResourceUsage.

2015-07-09 Thread Bartek Plotka
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36314/ --- (Updated July 9, 2015, 1:09 p.m.) Review request for mesos, Jie Yu, Niklas

Re: Review Request 36314: Added test for passing total slave's resources in ResourceUsage.

2015-07-09 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36314/#review91162 --- Ship it! LGTM! Some nits. src/tests/slave_tests.cpp (line 2258)

Re: Review Request 36314: Added test for passing total slave's resources in ResourceUsage.

2015-07-08 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36314/#review90923 --- Patch looks great! Reviews applied: [36204, 36314] All tests