----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40332/#review107692 -----------------------------------------------------------
src/master/allocator/mesos/hierarchical.cpp (line 416) <https://reviews.apache.org/r/40332/#comment166953> There might be new host added when recovery, so what about renaming "reregistered" to "added"? s/reregistered/added? src/master/allocator/mesos/hierarchical.cpp (line 993) <https://reviews.apache.org/r/40332/#comment166951> Shall we unify the tone in log files? What about s/Pausing/Paused src/master/allocator/mesos/hierarchical.cpp (line 1003) <https://reviews.apache.org/r/40332/#comment166952> s/Resuming/Resumed - Guangya Liu On 十一月 23, 2015, 11:35 p.m., Alexander Rukletsov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/40332/ > ----------------------------------------------------------- > > (Updated 十一月 23, 2015, 11:35 p.m.) > > > Review request for mesos, Bernd Mathiske, Joerg Schad, Joris Van Remoortere, > Joseph Wu, and Qian Zhang. > > > Bugs: MESOS-3981 > https://issues.apache.org/jira/browse/MESOS-3981 > > > Repository: mesos > > > Description > ------- > > See summary. > > > Diffs > ----- > > src/master/allocator/mesos/hierarchical.hpp > 2a21364fdcaa4ec5e5567b9f367c14a1579b9a49 > src/master/allocator/mesos/hierarchical.cpp > aee8ced1fbfec8cf30cb939ff67fadfc6b08f37a > > Diff: https://reviews.apache.org/r/40332/diff/ > > > Testing > ------- > > make check (Mac OS X 10.10.4) > > > Thanks, > > Alexander Rukletsov > >