> On Sept. 1, 2015, 6 p.m., Ben Mahler wrote: > > src/master/allocator/mesos/hierarchical.hpp, line 284 > > <https://reviews.apache.org/r/37958/diff/2/?file=1061155#file1061155line284> > > > > Can you move this back up to below 'initialized' for now?
Fixed. > On Sept. 1, 2015, 6 p.m., Ben Mahler wrote: > > include/mesos/master/allocator.hpp, line 81 > > <https://reviews.apache.org/r/37958/diff/2/?file=1061153#file1061153line81> > > > > How about s/allocationOptions/options/ here and everywhere else in this > > patch? Renamed to ```allocator::Options```. > On Sept. 1, 2015, 6 p.m., Ben Mahler wrote: > > docs/configuration.md, lines 291-295 > > <https://reviews.apache.org/r/37958/diff/2/?file=1061152#file1061152line291> > > > > Can you have this match the help message from flags.cpp? In particular, > > let's not tell people that this gives more predictable performance, as that > > is alluding to the fact that there is are performance bug(s) in the > > allocator, and we should fix those! Done. - James ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37958/#review97311 ----------------------------------------------------------- On Sept. 1, 2015, 4:34 p.m., James Peach wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37958/ > ----------------------------------------------------------- > > (Updated Sept. 1, 2015, 4:34 p.m.) > > > Review request for mesos and Ben Mahler. > > > Bugs: MESOS-3157 > https://issues.apache.org/jira/browse/MESOS-3157 > > > Repository: mesos > > > Description > ------- > > Normally, the mesos-master performs allocation cycles in response > to protocol messages and cluster events. This ties the cost of > allocation to the amount of churn in the environment, which can led > to unpredictable performance. Add an option to disable this, meaning > that allocations are only every performed by the batch interval > task. This makes resource allocation a fixed cost, independent of > the amount of environmental churn. > > > Diffs > ----- > > docs/configuration.md 2b23d48c8841e43e6e2776dfe6bfa7c022a941a7 > include/mesos/master/allocator.hpp 659f37b3f9d9fa02da9bdb6c85cd3c180a24b73a > src/master/allocator/mesos/allocator.hpp > aa55755a9c3250579e9366bdbc17a2449e95d659 > src/master/allocator/mesos/hierarchical.hpp > 38f8fd2c84314bb3731684d0e9795cb4f50a227e > src/master/flags.hpp e4b1df3f5a33049defff4688463274067f1f1ebf > src/master/flags.cpp 230c1dcedfbbb410b89defd90159d964c001a615 > src/master/master.cpp 56bcbcc08fa0f98416c5048080adb25efc588019 > src/tests/hierarchical_allocator_tests.cpp > 9748ca0b38888fee25dcec51c64d8ba84dbd4aaf > src/tests/mesos.hpp 906948d459b5a88a4ad7952801eb8c540b58c569 > > Diff: https://reviews.apache.org/r/37958/diff/ > > > Testing > ------- > > "make check" on CentOS 6 w/ devtoolset-3. Running in production for multiple > weeks. > > > Thanks, > > James Peach > >