Re: Review Request 31268: Wired up test allocator to allocator tests.

2015-04-20 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31268/ --- (Updated April 20, 2015, 1:51 p.m.) Review request for mesos, Kapil Arya,

Re: Review Request 31268: Wired up test allocator to allocator tests.

2015-04-20 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31268/#review80721 --- Patch looks great! Reviews applied: [31775, 31776, 33356, 31266,

Re: Review Request 31268: Wired up test allocator to allocator tests.

2015-04-15 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31268/ --- (Updated April 15, 2015, 2:22 p.m.) Review request for mesos, Kapil Arya,

Re: Review Request 31268: Wired up test allocator to allocator tests.

2015-04-15 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31268/#review80184 --- Bad patch! Reviews applied: [31775] Failed command:

Re: Review Request 31268: Wired up test allocator to allocator tests.

2015-04-07 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31268/ --- (Updated April 7, 2015, 12:49 p.m.) Review request for mesos, Kapil Arya,

Re: Review Request 31268: Wired up test allocator to allocator tests.

2015-04-07 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31268/#review79175 --- Patch looks great! Reviews applied: [31775, 31776, 31266, 31267,

Re: Review Request 31268: Wired up test allocator to allocator tests.

2015-04-02 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31268/#review78712 --- Ship it! Thanks for splitting these reviews in reviewable chunks!

Re: Review Request 31268: Wired up test allocator to allocator tests.

2015-04-01 Thread Alexander Rukletsov
On March 24, 2015, 2:49 p.m., Michael Park wrote: src/tests/master_allocator_tests.cpp, lines 123-126 https://reviews.apache.org/r/31268/diff/4/?file=903030#file903030line123 Could we indent this similar to this example from `src/tests/isolator_tests.cpp`: ```

Re: Review Request 31268: Wired up test allocator to allocator tests.

2015-04-01 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31268/#review78512 --- Ship it! Ship It! - Till Toenshoff On March 27, 2015, 4:27

Re: Review Request 31268: Wired up test allocator to allocator tests.

2015-04-01 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31268/ --- (Updated April 1, 2015, 2:29 p.m.) Review request for mesos, Kapil Arya,

Re: Review Request 31268: Wired up test allocator to allocator tests.

2015-04-01 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31268/#review78525 --- Patch looks great! Reviews applied: [31775, 31776, 31266, 31267,

Re: Review Request 31268: Wired up test allocator to allocator tests.

2015-03-27 Thread Michael Park
On March 24, 2015, 2:49 p.m., Michael Park wrote: src/tests/master_allocator_tests.cpp, lines 123-126 https://reviews.apache.org/r/31268/diff/4/?file=903030#file903030line123 Could we indent this similar to this example from `src/tests/isolator_tests.cpp`: ```

Re: Review Request 31268: Wired up test allocator to allocator tests.

2015-03-27 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31268/#review78074 --- Patch looks great! Reviews applied: [31775, 31776, 31266, 31267,

Re: Review Request 31268: Wired up test allocator to allocator tests.

2015-03-27 Thread Alexander Rukletsov
On March 24, 2015, 2:49 p.m., Michael Park wrote: src/tests/master_allocator_tests.cpp, lines 123-126 https://reviews.apache.org/r/31268/diff/4/?file=903030#file903030line123 Could we indent this similar to this example from `src/tests/isolator_tests.cpp`: ```

Re: Review Request 31268: Wired up test allocator to allocator tests.

2015-03-27 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31268/ --- (Updated March 27, 2015, 4:27 p.m.) Review request for mesos, Kapil Arya,

Re: Review Request 31268: Wired up test allocator to allocator tests.

2015-03-24 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31268/#review77572 --- src/tests/master_allocator_tests.cpp

Re: Review Request 31268: Wired up test allocator to allocator tests.

2015-03-23 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31268/ --- (Updated March 23, 2015, 2:27 p.m.) Review request for mesos, Kapil Arya,

Re: Review Request 31268: Wired up test allocator to allocator tests.

2015-03-23 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31268/#review77410 --- Patch looks great! Reviews applied: [31775, 31776, 31266, 31267,

Re: Review Request 31268: Wired up test allocator to allocator tests.

2015-03-13 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31268/ --- (Updated March 13, 2015, 9:51 p.m.) Review request for mesos, Kapil Arya,

Re: Review Request 31268: Wired up test allocator to allocator tests.

2015-03-13 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31268/#review76451 --- Patch looks great! Reviews applied: [31775, 31776, 31266, 31267,

Re: Review Request 31268: Wired up test allocator to allocator tests.

2015-03-05 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31268/ --- (Updated March 5, 2015, 9:07 p.m.) Review request for mesos, Kapil Arya,

Re: Review Request 31268: Wired up test allocator to allocator tests.

2015-03-05 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31268/#review75453 --- Patch looks great! Reviews applied: [31775, 31776, 31266, 31267,