----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5448/#review8547 -----------------------------------------------------------
Ship it! src/master/allocator_factory.cpp <https://reviews.apache.org/r/5448/#comment18204> We should probably ultimately eliminate this file. - Benjamin Hindman On June 22, 2012, 11:36 p.m., Thomas Marshall wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/5448/ > ----------------------------------------------------------- > > (Updated June 22, 2012, 11:36 p.m.) > > > Review request for mesos and Benjamin Hindman. > > > Description > ------- > > Renamed SimpleAllocator to DominantShareAllocator. > > > Diffs > ----- > > src/Makefile.am 8760f59 > src/local/local.cpp d35639f > src/master/allocator_factory.cpp e4f4969 > src/master/dominant_share_allocator.hpp PRE-CREATION > src/master/dominant_share_allocator.cpp PRE-CREATION > src/master/main.cpp bcbe35a > src/master/master.hpp 886f79c > src/master/simple_allocator.hpp 034b5d0 > src/master/simple_allocator.cpp 1c54feb > src/tests/exception_tests.cpp de7fcf3 > src/tests/fault_tolerance_tests.cpp b2529ca > src/tests/master_detector_tests.cpp 4cefbfa > src/tests/master_tests.cpp fcaf7dc > src/tests/resource_offers_tests.cpp c1f1760 > src/tests/slave_tests.cpp 84e8e7d > > Diff: https://reviews.apache.org/r/5448/diff/ > > > Testing > ------- > > > Thanks, > > Thomas Marshall > >
