Re: Review Request 51529: Initialize sorter pointers in HierarchicalAllocatorProcess constructor.

2016-08-30 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51529/#review147371 --- Patch looks great! Reviews applied: [51528, 51529] Passed comman

Re: Review Request 51529: Initialize sorter pointers in HierarchicalAllocatorProcess constructor.

2016-08-30 Thread Jiang Yan Xu
> On Aug. 30, 2016, 12:24 p.m., Benjamin Bannier wrote: > > Thanks for the review! > On Aug. 30, 2016, 12:24 p.m., Benjamin Bannier wrote: > > src/master/allocator/mesos/hierarchical.hpp, line 465 > > > > > > `s/s

Re: Review Request 51529: Initialize sorter pointers in HierarchicalAllocatorProcess constructor.

2016-08-30 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51529/#review147336 --- Fix it, then Ship it! src/master/allocator/mesos/hierarchical.

Review Request 51529: Initialize sorter pointers in HierarchicalAllocatorProcess constructor.

2016-08-30 Thread Jiang Yan Xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51529/ --- Review request for mesos, Alexander Rukletsov, Benjamin Bannier, Guangya Liu, an