Re: Review Request 35038: Updated slave to query resource estimator whenever it wants to forward an update.

2015-06-03 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35038/ --- (Updated June 4, 2015, 12:03 a.m.) Review request for mesos, Ben Mahler, Jie Yu

Re: Review Request 35038: Updated slave to query resource estimator whenever it wants to forward an update.

2015-06-03 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35038/#review86518 --- LGTM modulo Jie's comment on the hard check - Niklas Nielsen On J

Re: Review Request 35038: Updated slave to query resource estimator whenever it wants to forward an update.

2015-06-03 Thread Jie Yu
> On June 3, 2015, 11:44 p.m., Jie Yu wrote: > > Also, could you please adjust the comments above the public resource estimator 'oversubscribable()' interface since we incur a minimal interval in the slave? - Jie --- This is an automat

Re: Review Request 35038: Updated slave to query resource estimator whenever it wants to forward an update.

2015-06-03 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35038/#review86514 --- Ship it! src/slave/slave.cpp

Review Request 35038: Updated slave to query resource estimator whenever it wants to forward an update.

2015-06-03 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35038/ --- Review request for mesos, Ben Mahler, Jie Yu, and Niklas Nielsen. Bugs: MESOS-2