> On May 17, 2013, 7:12 p.m., Vinod Kone wrote: > > src/master/master.cpp, lines 1266-1272 > > <https://reviews.apache.org/r/11121/diff/2/?file=291857#file291857line1266> > > > > If we do decide to do this (I'm not completely convinced yet), we > > should do it in the allocator.
Care to elaborate? It looks like in HierarchicalAllocatorProcess::allocate() the offers are in a hash map, so I'm not sure how you could shuffle them there. - Brenden ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11121/#review20716 ----------------------------------------------------------- On June 6, 2013, 2:13 a.m., Brenden Matthews wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11121/ > ----------------------------------------------------------- > > (Updated June 6, 2013, 2:13 a.m.) > > > Review request for mesos. > > > Description > ------- > > Shuffle offers before sending to frameworks. > > This helps ensure fair allocation of resources. > > Review: https://reviews.apache.org/r/11121 > > > Diffs > ----- > > src/master/master.cpp a2e4b905f1ef5c00560917c133b27ac978988807 > > Diff: https://reviews.apache.org/r/11121/diff/ > > > Testing > ------- > > Used in production at airbnb. > > > Thanks, > > Brenden Matthews > >
