Re: Review Request 29925: Moved allocation related sources into a separate directory.

2015-02-10 Thread Alexander Rukletsov
On Feb. 5, 2015, 7:13 p.m., Vinod Kone wrote: src/Makefile.am, lines 466-469 https://reviews.apache.org/r/29925/diff/6/?file=848172#file848172line466 How about this structure for consistency: master/allocator/allocator.hpp master/allocator/mesos/hierarchical.hpp

Re: Review Request 29925: Moved allocation related sources into a separate directory.

2015-02-10 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29925/ --- (Updated Feb. 10, 2015, 7:59 p.m.) Review request for mesos, Bernd Mathiske,

Re: Review Request 29925: Moved allocation related sources into a separate directory.

2015-02-06 Thread Alexander Rukletsov
On Feb. 5, 2015, 7:13 p.m., Vinod Kone wrote: src/Makefile.am, lines 466-469 https://reviews.apache.org/r/29925/diff/6/?file=848172#file848172line466 How about this structure for consistency: master/allocator/allocator.hpp master/allocator/mesos/hierarchical.hpp

Re: Review Request 29925: Moved allocation related sources into a separate directory.

2015-02-05 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29925/#review71273 --- src/Makefile.am https://reviews.apache.org/r/29925/#comment116947

Re: Review Request 29925: Moved allocation related sources into a separate directory.

2015-02-04 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29925/ --- (Updated Feb. 4, 2015, 4:49 p.m.) Review request for mesos, Bernd Mathiske,

Re: Review Request 29925: Moved allocation related sources into a separate directory.

2015-01-30 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29925/ --- (Updated Jan. 30, 2015, 3:33 p.m.) Review request for mesos, Bernd Mathiske

Re: Review Request 29925: Moved allocation related sources into a separate directory.

2015-01-28 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29925/ --- (Updated Jan. 28, 2015, 5:28 p.m.) Review request for mesos, Bernd Mathiske

Re: Review Request 29925: Moved allocation related sources into a separate directory.

2015-01-27 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29925/#review69849 --- Ship it! Ship It! - Niklas Nielsen On Jan. 27, 2015, 5:22 a.m.,

Re: Review Request 29925: Moved allocation related sources into a separate directory.

2015-01-27 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29925/ --- (Updated Jan. 27, 2015, 1:22 p.m.) Review request for mesos, Bernd Mathiske

Re: Review Request 29925: Moved allocation related sources into a separate directory.

2015-01-22 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29925/#review69325 --- src/Makefile.am https://reviews.apache.org/r/29925/#comment113987

Re: Review Request 29925: Moved allocation related sources into a separate directory.

2015-01-20 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29925/ --- (Updated Jan. 20, 2015, 6:06 p.m.) Review request for mesos, Bernd Mathiske

Re: Review Request 29925: Moved allocation related sources into a separate directory.

2015-01-19 Thread Alexander Rukletsov
On Jan. 18, 2015, 1:11 p.m., Bernd Mathiske wrote: Why allocation/ and not master/allocation/? Are there allocators for slaves? Note that there is slave/containerizer/. In that vein it should be master/allocator/. Agreed. - Alexander

Re: Review Request 29925: Moved allocation related sources into a separate directory.

2015-01-18 Thread Bernd Mathiske
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29925/#review68562 --- Why allocation/ and not master/allocation/? Are there allocators

Re: Review Request 29925: Moved allocation related sources into a separate directory.

2015-01-16 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29925/#review68465 --- Ship it! Ship It! - Niklas Nielsen On Jan. 15, 2015, 8:58 a.m.,

Review Request 29925: Moved allocation related sources into a separate directory.

2015-01-15 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29925/ --- Review request for mesos, Bernd Mathiske and Niklas Nielsen. Bugs: MESOS-2213