> On Aug. 1, 2014, 4:30 p.m., Jie Yu wrote: > > src/master/master.cpp, line 3434 > > <https://reviews.apache.org/r/24128/diff/4/?file=648800#file648800line3434> > > > > Could you please add an optional filer for 'resourcesRecovered' > > (similar to that in 'resourcesUnused'), and use a default filter here.
I'm not sure how necessary this is, and I'm aware of scope creep here. I've made the change but I think this would be better split out. Also added a TODO to merge the similar resources* methods on the Allocator. - Dominic ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24128/#review49400 ----------------------------------------------------------- On Aug. 1, 2014, 5:03 p.m., Dominic Hamon wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24128/ > ----------------------------------------------------------- > > (Updated Aug. 1, 2014, 5:03 p.m.) > > > Review request for mesos, Benjamin Hindman and Jie Yu. > > > Bugs: MESOS-1654 > https://issues.apache.org/jira/browse/MESOS-1654 > > > Repository: mesos-git > > > Description > ------- > > when network isolation is enabled, we should keep track of how many executors > have launched on a given host to avoid sending offers for those that have hit > the container limit. > > this is a short-term fix for the issue to enable broader testing of network > monitoring and isolation. > > > Diffs > ----- > > docs/allocation-module.md b308a23c0cdaf9bcb31fc0b13f584cd04cfa1aaa > src/master/allocator.hpp be6b038a9db076fb26cf40bfb61181f0fa860938 > src/master/flags.hpp 2b92de37b64c3363915918c65ab092efd5b378d6 > src/master/hierarchical_allocator_process.hpp > 35d1579bbd665dfdd238932902ae16880dadeb66 > src/master/master.cpp 77f2536c36df9a0a1f6021bd4732baa3ab424459 > src/tests/allocator_tests.cpp b920533bbc36a154c02b467673bc7a0d50d65bc7 > src/tests/master_tests.cpp 5a1cf7f1f7a4864f516825cc14504c870b60055e > src/tests/mesos.hpp 75c66fda2485afa0d4541e710780d90b3411839a > src/tests/slave_recovery_tests.cpp 0b17a8afb6aa66ab19f6fcd90906fb4eb2c39f05 > > Diff: https://reviews.apache.org/r/24128/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Dominic Hamon > >