Re: Review Request 31262: Moved allocator actions before TestAllocator.

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

Re: Review Request 31262: Moved allocator actions before TestAllocator.

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

Re: Review Request 31262: Moved allocator actions before TestAllocator.

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

Re: Review Request 31262: Moved allocator actions before TestAllocator.

2015-04-02 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31262/#review78696 --- Ship it! Ship It! - Vinod Kone On April 1, 2015, 2:27 p.m., Alex

Re: Review Request 31262: Moved allocator actions before TestAllocator.

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

Re: Review Request 31262: Moved allocator actions before TestAllocator.

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

Re: Review Request 31262: Moved allocator actions before TestAllocator.

2015-03-23 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31262/#review77401 --- Ship it! Ship It! - Kapil Arya On March 23, 2015, 10:06 a.m., Al

Re: Review Request 31262: Moved allocator actions before TestAllocator.

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

Re: Review Request 31262: Moved allocator actions before TestAllocator.

2015-03-18 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31262/ --- (Updated March 19, 2015, 12:36 a.m.) Review request for mesos, Kapil Arya, Mich

Re: Review Request 31262: Moved allocator actions before TestAllocator.

2015-03-18 Thread Alexander Rukletsov
> On March 17, 2015, 9:06 p.m., Kapil Arya wrote: > > SGTM minus the following change in the description: > > s/before the are used/before they are used/ thanks! - Alexander --- This is an automatically generated e-mail. To reply, visit

Re: Review Request 31262: Moved allocator actions before TestAllocator.

2015-03-17 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31262/#review76811 --- SGTM minus the following change in the description: s/before the are

Re: Review Request 31262: Moved allocator actions before TestAllocator.

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

Re: Review Request 31262: Moved allocator actions before TestAllocator.

2015-03-13 Thread Alexander Rukletsov
> On March 10, 2015, 9:10 p.m., Niklas Nielsen wrote: > > Can you leave a comment about why you need it? (In terms of your following > > refactorings will break if you don't do it) As I mention in the description, once TestAllocator is not a template (r/31263/), the allocator actions should be

Re: Review Request 31262: Moved allocator actions before TestAllocator.

2015-03-10 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31262/#review75964 --- Can you leave a comment about why you need it? (In terms of your fol

Re: Review Request 31262: Moved allocator actions before TestAllocator.

2015-03-05 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31262/#review75412 --- Ship it! Ship It! - Joerg Schad On March 5, 2015, 9:05 p.m., Ale

Re: Review Request 31262: Moved allocator actions before TestAllocator.

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