Re: Review Request 41042: Added description of the LoadQoSController in the oversubscription.md

2016-01-05 Thread Bartek Plotka
> On Jan. 4, 2016, 6:35 p.m., Vinod Kone wrote: > > docs/oversubscription.md, line 204 > > > > > > s/controller/the `load` qos controller/ > > Guangya Liu wrote: > s/controller/, the load qos controller/?

Re: Review Request 41042: Added description of the LoadQoSController in the oversubscription.md

2016-01-05 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41042/#review112850 --- Bad patch! Reviews applied: [40617] Failed command:

Re: Review Request 41042: Added description of the LoadQoSController in the oversubscription.md

2016-01-05 Thread Bartek Plotka
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41042/ --- (Updated Jan. 5, 2016, 10:41 a.m.) Review request for mesos, Niklas Nielsen

Re: Review Request 41042: Added description of the LoadQoSController in the oversubscription.md

2016-01-05 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41042/#review112831 --- Ship it! Ship It! - Guangya Liu On 一月 5, 2016, 10:41 a.m.,

Re: Review Request 41042: Added description of the LoadQoSController in the oversubscription.md

2016-01-04 Thread Guangya Liu
> On 一月 4, 2016, 6:35 p.m., Vinod Kone wrote: > > docs/oversubscription.md, line 204 > > > > > > s/controller/the `load` qos controller/ s/controller/, the load qos controller/? - Guangya

Re: Review Request 41042: Added description of the LoadQoSController in the oversubscription.md

2016-01-04 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41042/#review112612 --- docs/oversubscription.md (line 202)

Re: Review Request 41042: Added description of the LoadQoSController in the oversubscription.md

2015-12-08 Thread Bartek Plotka
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41042/ --- (Updated Dec. 8, 2015, 1:24 p.m.) Review request for mesos, Niklas Nielsen and

Re: Review Request 41042: Added description of the LoadQoSController in the oversubscription.md

2015-12-08 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41042/#review109343 --- Patch looks great! Reviews applied: [40617, 41042] Passed

Re: Review Request 41042: Added description of the LoadQoSController in the oversubscription.md

2015-12-07 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41042/#review109167 --- Ship it! docs/oversubscription.md (line 201)

Re: Review Request 41042: Added description of the LoadQoSController in the oversubscription.md

2015-12-07 Thread Bartek Plotka
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41042/ --- (Updated Dec. 7, 2015, 9:18 a.m.) Review request for mesos, Niklas Nielsen and