Re: Review Request 31266: Added support for allocator modules.

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

Re: Review Request 31266: Added support for allocator modules.

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

Re: Review Request 31266: Added support for allocator modules.

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

Re: Review Request 31266: Added support for allocator modules.

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

Re: Review Request 31266: Added support for allocator modules.

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

Re: Review Request 31266: Added support for allocator modules.

2015-03-27 Thread Alexander Rukletsov
On March 24, 2015, 5:55 a.m., Michael Park wrote: Michael Park wrote: Hm, upon looking at other files in `include/mesos/module/` it looks like the 2 style issues I mentioned below is apparent in all of them. Is there a reason for this? or shoud they be fixed as a batch in a

Re: Review Request 31266: Added support for allocator modules.

2015-03-27 Thread Michael Park
On March 24, 2015, 5:55 a.m., Michael Park wrote: Michael Park wrote: Hm, upon looking at other files in `include/mesos/module/` it looks like the 2 style issues I mentioned below is apparent in all of them. Is there a reason for this? or shoud they be fixed as a batch in a

Re: Review Request 31266: Added support for allocator modules.

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

Re: Review Request 31266: Added support for allocator modules.

2015-03-24 Thread Alexander Rojas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31266/#review77587 --- Ship it! Ship It! - Alexander Rojas On March 23, 2015, 3:03

Re: Review Request 31266: Added support for allocator modules.

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

Re: Review Request 31266: Added support for allocator modules.

2015-03-23 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31266/#review77528 --- include/mesos/module/allocator.hpp

Re: Review Request 31266: Added support for allocator modules.

2015-03-23 Thread Michael Park
On March 24, 2015, 5:55 a.m., Michael Park wrote: Hm, upon looking at other files in `include/mesos/module/` it looks like the 2 style issues I mentioned below is apparent in all of them. Is there a reason for this? or shoud they be fixed as a batch in a separate review request? -

Re: Review Request 31266: Added support for allocator modules.

2015-03-17 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31266/#review76808 --- Ship it! Ship It! - Kapil Arya On March 13, 2015, 5:44 p.m.,

Re: Review Request 31266: Added support for allocator modules.

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

Re: Review Request 31266: Added support for allocator modules.

2015-03-10 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31266/#review75962 --- Ship it! - Niklas Nielsen On March 5, 2015, 1:05 p.m., Alexander

Re: Review Request 31266: Added support for allocator modules.

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