Re: Review Request 35947: Added a new API call 'updateAvailable' to the allocator.

2015-07-28 Thread Alexander Rukletsov
> On July 8, 2015, 5:59 p.m., Alexander Rukletsov wrote: > > include/mesos/master/allocator.hpp, lines 133-135 > > > > > > And we introduce a libprocess dependency into `Allocator` interface. I > > think it's a promin

Re: Review Request 35947: Added a new API call 'updateAvailable' to the allocator.

2015-07-24 Thread Michael Park
> On July 8, 2015, 5:59 p.m., Alexander Rukletsov wrote: > > include/mesos/master/allocator.hpp, lines 133-135 > > > > > > And we introduce a libprocess dependency into `Allocator` interface. I > > think it's a promin

Re: Review Request 35947: Added a new API call 'updateAvailable' to the allocator.

2015-07-24 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35947/ --- (Updated July 24, 2015, 9:26 p.m.) Review request for mesos, Alexander Rukletso

Re: Review Request 35947: Added a new API call 'updateAvailable' to the allocator.

2015-07-24 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35947/ --- (Updated July 24, 2015, 3:50 p.m.) Review request for mesos, Alexander Rukletso

Re: Review Request 35947: Added a new API call 'updateAvailable' to the allocator.

2015-07-09 Thread Alexander Rukletsov
> On July 8, 2015, 5:59 p.m., Alexander Rukletsov wrote: > > include/mesos/master/allocator.hpp, lines 133-135 > > > > > > And we introduce a libprocess dependency into `Allocator` interface. I > > think it's a promin

Re: Review Request 35947: Added a new API call 'updateAvailable' to the allocator.

2015-07-09 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35947/#review91081 --- src/tests/hierarchical_allocator_tests.cpp (line 785)

Re: Review Request 35947: Added a new API call 'updateAvailable' to the allocator.

2015-07-08 Thread Jie Yu
> On July 8, 2015, 5:59 p.m., Alexander Rukletsov wrote: > > include/mesos/master/allocator.hpp, lines 133-135 > > > > > > And we introduce a libprocess dependency into `Allocator` interface. I > > think it's a promin

Re: Review Request 35947: Added a new API call 'updateAvailable' to the allocator.

2015-07-08 Thread Alexander Rukletsov
> On July 8, 2015, 5:37 p.m., Alexander Rukletsov wrote: > > A high-level concern I would like to share with you guys, though it isn't > > directly related to this particular patch. I have a feeling that we don't > > really care about keeping the Allocator interface neat, brief, and concise, >

Re: Review Request 35947: Added a new API call 'updateAvailable' to the allocator.

2015-07-08 Thread Jie Yu
> On July 8, 2015, 5:37 p.m., Alexander Rukletsov wrote: > > A high-level concern I would like to share with you guys, though it isn't > > directly related to this particular patch. I have a feeling that we don't > > really care about keeping the Allocator interface neat, brief, and concise, >

Re: Review Request 35947: Added a new API call 'updateAvailable' to the allocator.

2015-07-08 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35947/#review90936 --- include/mesos/master/allocator.hpp (lines 133 - 135)

Re: Review Request 35947: Added a new API call 'updateAvailable' to the allocator.

2015-07-08 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35947/#review90930 --- A high-level concern I would like to share with you guys, though it

Re: Review Request 35947: Added a new API call 'updateAvailable' to the allocator.

2015-07-07 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35947/#review90839 --- Ship it! src/master/allocator/mesos/hierarchical.hpp (lines 734 -

Review Request 35947: Added a new API call 'updateAvailable' to the allocator.

2015-06-26 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35947/ --- Review request for mesos, Alexander Rukletsov, Benjamin Hindman, Ben Mahler, and