Review Request 34631: Added QoS Controller.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34631/ --- Review request for mesos. Repository: mesos Description --- Added QoS Co

Re: Review Request 34631: Added QoS Controller.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34631/#review85150 --- include/mesos/slave/qos_controller.hpp

Re: Review Request 34631: Added QoS Controller.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34631/ --- (Updated May 26, 2015, 8:30 a.m.) Review request for mesos, Bartek Plotka, Jie

Re: Review Request 34631: Added QoS Controller.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34631/#review85267 --- Ship it! Ship It! - Bartek Plotka On May 26, 2015, 3:30 p.m., Ni

Re: Review Request 34631: Added QoS Controller.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34631/#review85757 --- include/mesos/slave/qos_controller.hpp

Re: Review Request 34631: Added QoS Controller.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34631/#review86004 --- Ship it! Ditto Vinod's comment on s/BE/revokable/ src/slave/qos_c

Re: Review Request 34631: Added QoS Controller.

> On May 29, 2015, 5:39 p.m., Vinod Kone wrote: > > include/mesos/slave/qos_controller.hpp, line 62 > > > > > > s/QoSCorrection/list/ ? Probably calling it 'correction' and returns a single QoSCorrection? Can you remi

Re: Review Request 34631: Added QoS Controller.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34631/ --- (Updated June 2, 2015, 9:26 p.m.) Review request for mesos, Bartek Plotka, Jie

Re: Review Request 34631: Added QoS Controller.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34631/#review86447 --- include/mesos/slave/qos_controller.hpp

Re: Review Request 34631: Added QoS Controller.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34631/#review86499 --- Ship it! include/mesos/slave/qos_controller.hpp

Re: Review Request 34631: Added QoS Controller.

> On June 3, 2015, 10:54 a.m., Bartek Plotka wrote: > > include/mesos/slave/qos_controller.hpp, line 52 > > > > > > Small thing: > > s/type/name/ ..to be consistent with allocator factory. (: > > > > What i

Re: Review Request 34631: Added QoS Controller.

> On June 3, 2015, 5:54 p.m., Bartek Plotka wrote: > > include/mesos/slave/qos_controller.hpp, line 52 > > > > > > Small thing: > > s/type/name/ ..to be consistent with allocator factory. (: > > > > What is

Re: Review Request 34631: Added QoS Controller.

> On June 3, 2015, 10:54 a.m., Bartek Plotka wrote: > > include/mesos/slave/qos_controller.hpp, line 52 > > > > > > Small thing: > > s/type/name/ ..to be consistent with allocator factory. (: > > > > What i

Re: Review Request 34631: Added QoS Controller.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34631/ --- (Updated June 4, 2015, 10:42 a.m.) Review request for mesos, Bartek Plotka, Jie

Re: Review Request 34631: Added QoS Controller.

> On June 3, 2015, 4:02 p.m., Jie Yu wrote: > > include/mesos/slave/qos_controller.hpp, lines 25-26 > > > > > > Can you add a wrapper header > > mesos/slave/oversubscription.hpp > > > > similar to what we d

Re: Review Request 34631: Added QoS Controller.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34631/ --- (Updated June 4, 2015, 2:23 p.m.) Review request for mesos, Bartek Plotka, Jie