Re: Review Request 34730: Added 'updateSlave()' in master to handle oversubscribed resources.

2015-05-28 Thread Vinod Kone
> On May 28, 2015, 12:02 a.m., Niklas Nielsen wrote: > > src/master/master.cpp, lines 3469-3470 > > > > > > Was there some additional math, only to rescind if the estimate is > > lower? If not now, should we add a to

Re: Review Request 34730: Added 'updateSlave()' in master to handle oversubscribed resources.

2015-05-27 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34730/#review85474 --- Ship it! Ship It! - Jie Yu On May 27, 2015, 11:11 p.m., Vinod Ko

Re: Review Request 34730: Added 'updateSlave()' in master to handle oversubscribed resources.

2015-05-27 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34730/#review85468 --- Ship it! Taken it is a stub patch src/master/master.cpp