Re: Review Request 49610: Added test case for quota allocation with reserved resources.

2016-07-11 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49610/ --- (Updated July 11, 2016, 8:47 a.m.) Review request for mesos and Alexander

Re: Review Request 49610: Added test case for quota allocation with reserved resources.

2016-07-06 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49610/#review141008 --- Fix it, then Ship it! Thanks a lot for the test Neil! It looks

Re: Review Request 49610: Added test case for quota allocation with reserved resources.

2016-07-05 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49610/ --- (Updated July 5, 2016, 8:51 a.m.) Review request for mesos and Alexander

Re: Review Request 49610: Added test case for quota allocation with reserved resources.

2016-07-05 Thread Neil Conway
> On July 4, 2016, 11:29 p.m., Guangya Liu wrote: > > src/tests/hierarchical_allocator_tests.cpp, lines 3208-3218 > > > > > > Does there are any special reason that you are using `dynamic > > reservation` here?

Re: Review Request 49610: Added test case for quota allocation with reserved resources.

2016-07-04 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49610/#review140704 --- src/tests/hierarchical_allocator_tests.cpp (line 3192)

Review Request 49610: Added test case for quota allocation with reserved resources.

2016-07-04 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49610/ --- Review request for mesos and Alexander Rukletsov. Repository: mesos