Re: Review Request 34545: Updated the allocator related docs.

2015-05-29 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34545/ --- (Updated May 29, 2015, 3:39 p.m.) Review request for mesos, Adam B, Bernd Mathi

Re: Review Request 34545: Updated the allocator related docs.

2015-05-29 Thread Alexander Rukletsov
> On May 29, 2015, 3:01 p.m., Niklas Nielsen wrote: > > docs/allocation-module.md, line 101 > > > > > > s/several/two/? several is less error-prone: one we get more steps, this sentence should not be updated. - Ale

Re: Review Request 34545: Updated the allocator related docs.

2015-05-29 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34545/#review85723 --- Ship it! docs/allocation-module.md

Re: Review Request 34545: Updated the allocator related docs.

2015-05-29 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34545/#review85716 --- Patch looks great! Reviews applied: [34545] All tests passed. - M

Re: Review Request 34545: Updated the allocator related docs.

2015-05-29 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34545/ --- (Updated May 29, 2015, 12:48 p.m.) Review request for mesos, Adam B, Bernd Math

Re: Review Request 34545: Updated the allocator related docs.

2015-05-29 Thread Alexander Rukletsov
> On May 29, 2015, 9:01 a.m., Adam B wrote: > > docs/allocation-module.md, line 7 > > > > > > technically, the "default" is "hierarchical DRF" (role, then framework). Indeed, good point! > On May 29, 2015, 9:01 a.m.,

Re: Review Request 34545: Updated the allocator related docs.

2015-05-29 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34545/#review85704 --- Patch looks great! Reviews applied: [34545] All tests passed. - M

Re: Review Request 34545: Updated the allocator related docs.

2015-05-29 Thread Adam B
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34545/#review85693 --- Looks good. Just some wording suggestions. docs/allocation-module.

Re: Review Request 34545: Updated the allocator related docs.

2015-05-26 Thread Bernd Mathiske
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34545/#review85190 --- Ship it! Ship It! - Bernd Mathiske On May 26, 2015, 3:37 a.m., A

Re: Review Request 34545: Updated the allocator related docs.

2015-05-26 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34545/ --- (Updated May 26, 2015, 10:37 a.m.) Review request for mesos, Adam B, Bernd Math

Re: Review Request 34545: Updated the allocator related docs.

2015-05-26 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34545/ --- (Updated May 26, 2015, 9:57 a.m.) Review request for mesos, Adam B, Bernd Mathi

Re: Review Request 34545: Updated the allocator related docs.

2015-05-22 Thread Bernd Mathiske
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34545/#review84903 --- docs/allocation-module.md

Re: Review Request 34545: Updated the allocator related docs.

2015-05-21 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34545/#review84711 --- Patch looks great! Reviews applied: [34545] All tests passed. - M

Re: Review Request 34545: Updated the allocator related docs.

2015-05-21 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34545/ --- (Updated May 21, 2015, 2:58 p.m.) Review request for mesos, Adam B, Bernd Mathi