----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6097/#review9374 -----------------------------------------------------------
Ship it! src/master/allocator.hpp <https://reviews.apache.org/r/6097/#comment20109> We should really have docs for this interface functions. Since many more allocators are going to be written against this. src/master/dominant_share_allocator.cpp <https://reviews.apache.org/r/6097/#comment20110> CHECK(initialized) src/master/dominant_share_allocator.cpp <https://reviews.apache.org/r/6097/#comment20111> new line - Vinod Kone On July 23, 2012, 10:10 p.m., Benjamin Hindman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/6097/ > ----------------------------------------------------------- > > (Updated July 23, 2012, 10:10 p.m.) > > > Review request for mesos, Thomas Marshall and Vinod Kone. > > > Description > ------- > > See summary. > > > This addresses bug MESOS-239. > https://issues.apache.org/jira/browse/MESOS-239 > > > Diffs > ----- > > src/master/allocator.hpp e55b9ec > src/master/dominant_share_allocator.hpp bc5505b > src/master/dominant_share_allocator.cpp a2de26f > src/master/master.cpp 43407fa > src/tests/utils.hpp ced8592 > > Diff: https://reviews.apache.org/r/6097/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Benjamin Hindman > >
