Re: Review Request 37908: Silence oversubscription logging

2015-09-01 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37908/#review97351 --- Ship it! Ship It! - Vinod Kone On Aug. 31, 2015, 6:28 p.m., Nikl

Re: Review Request 37908: Silence oversubscription logging

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

Re: Review Request 37908: Silence oversubscription logging

2015-08-31 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37908/ --- (Updated Aug. 31, 2015, 11:28 a.m.) Review request for mesos and Jie Yu. Chan

Re: Review Request 37908: Silence oversubscription logging

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

Re: Review Request 37908: Silence oversubscription logging

2015-08-28 Thread Niklas Nielsen
> On Aug. 28, 2015, 3:42 p.m., Jie Yu wrote: > > src/slave/slave.cpp, lines 4240-4248 > > > > > > Hum, why zero is a special case here? What if the RE initially returns > > 10cpus, and due to some reaons, it return

Re: Review Request 37908: Silence oversubscription logging

2015-08-28 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37908/#review96940 --- src/slave/slave.cpp (line 4226)

Review Request 37908: Silence oversubscription logging

2015-08-28 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37908/ --- Review request for mesos and Jie Yu. Repository: mesos Description --- R