> On July 24, 2019, 9:34 p.m., Jiang Yan Xu wrote: > > include/mesos/allocator/allocator.hpp > > Lines 66 (patched) > > <https://reviews.apache.org/r/71080/diff/7/?file=2156698#file2156698line66> > > > > `maxCompletedFrameworks` is used for the metrics to keep for completed > > frameworks in the allocator. > > > > `maxCompletedFramework_ids` (we should name it > > `maxCompletedFrameworkIDs` if it were useful) is not used and we don't need > > to add this field.
maxCompletedFramework_ids can be removed since we don't need it in metrics - Xudong ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71080/#review216836 ----------------------------------------------------------- On July 24, 2019, 11:10 p.m., Xudong Ni wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71080/ > ----------------------------------------------------------- > > (Updated July 24, 2019, 11:10 p.m.) > > > Review request for mesos, James Peach and Jiang Yan Xu. > > > Bugs: MESOS-8636 > https://issues.apache.org/jira/browse/MESOS-8636 > > > Repository: mesos > > > Description > ------- > > It should be separately from that for webUI and endpoints. > Currently the master stores the history of completed frameworks in > a map with the full historical data of the framework, it is > prohibitively expensive to keep a long history; In order to reject > frameworks from reregistering if they have previously marked as > completed, we only need to persist the framework IDs and are able > to keep long history. > > > Diffs > ----- > > docs/configuration/master.md c56ac8510ea968f9587e23e81ed310caa968ee9e > docs/operator-http-api.md 1167838928e0fcb774cf904fa6ef5e6847134e15 > src/master/constants.hpp 26afa356b7844b4ec6c2caeef33bd39c51148d5f > src/master/flags.hpp 78623d68bf428cd3f52684303d98a525d42eb878 > src/master/flags.cpp 74f4daadd48e8e691be43759b88dc8b3c2df489a > src/master/http.cpp 765d5052ab6a8d731113f8922d20fb280b843003 > src/master/master.hpp 5c229c53f96dda6b30731017b8adbf57c9fc9f95 > src/master/master.cpp fbde112eabf2c6da363ab313cf3f4c54f581eaa7 > > > Diff: https://reviews.apache.org/r/71080/diff/8/ > > > Testing > ------- > > make check > > sudo GLOG_v=1 ./bin/mesos-tests.sh --verbose > --gtest_filter=MasterTest.MaxCompletedFrameworksFlag --gtest_break_on_failure > --gtest_repeat=1000 > [ OK ] MasterTest.MaxCompletedFrameworksFlag (230 ms) > [----------] 1 test from MasterTest (235 ms total) > > [----------] Global test environment tear-down > [==========] 1 test from 1 test case ran. (281 ms total) > [ PASSED ] 1 test. > > > Thanks, > > Xudong Ni > >