Re: Review Request 43588: Added allocator recovery tests in presence of quota.

2016-04-19 Thread Klaus Ma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43588/#review129691 --- ping :). - Klaus Ma On Feb. 23, 2016, 7:55 a.m., Joerg Schad wr

Re: Review Request 43588: Added allocator recovery tests in presence of quota.

2016-02-22 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43588/#review120270 --- Patch looks great! Reviews applied: [43588] Passed command: expo

Re: Review Request 43588: Added allocator recovery tests in presence of quota.

2016-02-22 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43588/ --- (Updated Feb. 22, 2016, 11:55 p.m.) Review request for mesos, Alexander Ruklets

Re: Review Request 43588: Added allocator recovery tests in presence of quota.

2016-02-22 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43588/ --- (Updated Feb. 22, 2016, 11:51 p.m.) Review request for mesos, Alexander Ruklets

Re: Review Request 43588: Added allocator recovery tests in presence of quota.

2016-02-22 Thread Joerg Schad
> On Feb. 16, 2016, 3:28 a.m., Klaus Ma wrote: > > src/tests/hierarchical_allocator_tests.cpp, line 2417 > > > > > > No allocation because there is no slaves. We have trigger allocation by > > `Clock::advance(flags

Re: Review Request 43588: Added allocator recovery tests in presence of quota.

2016-02-15 Thread Klaus Ma
> On Feb. 16, 2016, 11:28 a.m., Klaus Ma wrote: > > src/tests/hierarchical_allocator_tests.cpp, line 2400 > > > > > > Move to HierarchicalAllocatorTest. > > Joerg Schad wrote: > This will be done by/after https

Re: Review Request 43588: Added allocator recovery tests in presence of quota.

2016-02-15 Thread Joerg Schad
> On Feb. 16, 2016, 3:28 a.m., Klaus Ma wrote: > > src/tests/hierarchical_allocator_tests.cpp, line 2400 > > > > > > Move to HierarchicalAllocatorTest. This will be done by/after https://reviews.apache.org/r/41950.

Re: Review Request 43588: Added allocator recovery tests in presence of quota.

2016-02-15 Thread Klaus Ma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43588/#review119257 --- src/tests/hierarchical_allocator_tests.cpp (line 2400)

Re: Review Request 43588: Added allocator recovery tests in presence of quota.

2016-02-15 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43588/#review119255 --- src/tests/hierarchical_allocator_tests.cpp (line 2390)

Re: Review Request 43588: Added allocator recovery tests in presence of quota.

2016-02-15 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43588/#review119252 --- Patch looks great! Reviews applied: [43588] Passed command: expo