----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11121/#review20716 -----------------------------------------------------------
src/master/master.cpp <https://reviews.apache.org/r/11121/#comment42774> put it above line #19, so that it is in alphabetical order. src/master/master.cpp <https://reviews.apache.org/r/11121/#comment42779> If we do decide to do this (I'm not completely convinced yet), we should do it in the allocator. - Vinod Kone On May 14, 2013, 5:57 p.m., Brenden Matthews wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11121/ > ----------------------------------------------------------- > > (Updated May 14, 2013, 5:57 p.m.) > > > Review request for mesos. > > > Description > ------- > > From 085b679064dfdad9c135db47556b5bc1820e23f1 Mon Sep 17 00:00:00 2001 > From: Brenden Matthews <[email protected]> > Date: Sat, 27 Apr 2013 09:58:46 -0700 > Subject: [PATCH 14/24] Shuffle offers before sending to frameworks. > > This helps ensure fair allocation of resources. > > Review: https://reviews.apache.org/r/11121 > --- > src/master/master.cpp | 8 ++++++++ > 1 file changed, 8 insertions(+) > > > Diffs > ----- > > src/master/master.cpp 3207157961e772ae543a236536ae1559acfe110b > > Diff: https://reviews.apache.org/r/11121/diff/ > > > Testing > ------- > > Used in production at airbnb. > > > Thanks, > > Brenden Matthews > >
