Re: Review Request 42672: Explicitly checked for the absence of allocation.

2016-02-09 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42672/#review118420 --- Ship it! Ship It! - Ben Mahler On Jan. 28, 2016, 1:22 p.m.,

Re: Review Request 42672: Explicitly checked for the absence of allocation.

2016-01-28 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42672/ --- (Updated Jan. 28, 2016, 1:22 p.m.) Review request for mesos and Ben Mahler. C

Re: Review Request 42672: Explicitly checked for the absence of allocation.

2016-01-28 Thread Alexander Rukletsov
> On Jan. 23, 2016, 6:31 a.m., Guangya Liu wrote: > > src/tests/hierarchical_allocator_tests.cpp, line 1494 > > > > > > remove the blank line? Which line do you mean? - Alexander ---

Re: Review Request 42672: Explicitly checked for the absence of allocation.

2016-01-23 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42672/#review115980 --- Patch looks great! Reviews applied: [42633, 42636, 42657, 42658,

Re: Review Request 42672: Explicitly checked for the absence of allocation.

2016-01-22 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42672/#review115975 --- src/tests/hierarchical_allocator_tests.cpp (line 1480)

Review Request 42672: Explicitly checked for the absence of allocation.

2016-01-22 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42672/ --- Review request for mesos and Ben Mahler. Repository: mesos Description --