Re: Review Request 36488: Added oversubscription user doc.

2015-07-17 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36488/ --- (Updated July 17, 2015, 12:47 p.m.) Review request for mesos, Adam B and Jie

Re: Review Request 36488: Added oversubscription user doc.

2015-07-17 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36488/#review92126 --- Bad patch! Reviews applied: [36488] Failed command:

Re: Review Request 36488: Added oversubscription user doc.

2015-07-17 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36488/ --- (Updated July 17, 2015, 3:27 p.m.) Review request for mesos, Adam B and Jie

Re: Review Request 36488: Added oversubscription user doc.

2015-07-17 Thread Niklas Nielsen
On July 17, 2015, 1:37 p.m., Adam B wrote: docs/oversubscription.md, lines 99-104 https://reviews.apache.org/r/36488/diff/2-5/?file=1011899#file1011899line99 Did this removed text go somewhere else? I can't find it anymore. Seemed useful. Jie pointed out that the allocator

Re: Review Request 36488: Added oversubscription user doc.

2015-07-17 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36488/ --- (Updated July 17, 2015, 2:27 p.m.) Review request for mesos, Adam B and Jie

Re: Review Request 36488: Added oversubscription user doc.

2015-07-17 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36488/ --- (Updated July 17, 2015, 1 p.m.) Review request for mesos, Adam B and Jie Yu.

Re: Review Request 36488: Added oversubscription user doc.

2015-07-17 Thread Adam B
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36488/#review92122 --- Ship it! I can't see the jpg to see how you changed it, but the

Re: Review Request 36488: Added oversubscription user doc.

2015-07-17 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36488/#review92132 --- Ship it! LGTM! Thanks Niklas! docs/oversubscription.md (line 35)

Re: Review Request 36488: Added oversubscription user doc.

2015-07-17 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36488/ --- (Updated July 17, 2015, 3:19 p.m.) Review request for mesos, Adam B and Jie

Re: Review Request 36488: Added oversubscription user doc.

2015-07-16 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36488/#review91988 --- Looks good Nik! docs/oversubscription.md (line 6)

Re: Review Request 36488: Added oversubscription user doc.

2015-07-16 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36488/ --- (Updated July 16, 2015, 3:39 p.m.) Review request for mesos, Adam B and Jie

Re: Review Request 36488: Added oversubscription user doc.

2015-07-16 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36488/#review91984 --- Bad patch! Reviews applied: [36488] Failed command:

Re: Review Request 36488: Added oversubscription user doc.

2015-07-15 Thread Adam B
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36488/#review91840 --- Looks great! Helped me understand the feature much better. Comments

Re: Review Request 36488: Added oversubscription user doc.

2015-07-14 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36488/ --- (Updated July 14, 2015, 5:08 p.m.) Review request for mesos, Adam B and Jie

Re: Review Request 36488: Added oversubscription user doc.

2015-07-14 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36488/#review91696 --- Bad patch! Reviews applied: [36488] Failed command: