> On May 17, 2013, 6:43 p.m., Vinod Kone wrote: > > hadoop/mesos/src/java/org/apache/hadoop/mapred/MesosScheduler.java, lines > > 332-335 > > <https://reviews.apache.org/r/11117/diff/1/?file=290977#file290977line332> > > > > Why did you have to do this here? > > > > I would imagine the underlying scheduler would take care of this > > appropriately? > > > > Did you see any specific issue that you are trying to fix?
The list of trackers also contains the unhealthy (i.e., blacklisted) trackers. This excludes those trackers from slot calculation. - Brenden ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11117/#review20712 ----------------------------------------------------------- On May 14, 2013, 12:20 a.m., Brenden Matthews wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11117/ > ----------------------------------------------------------- > > (Updated May 14, 2013, 12:20 a.m.) > > > Review request for mesos. > > > Description > ------- > > From bf98ddc157c26c2653160d378bac2a1cc7f09a2b Mon Sep 17 00:00:00 2001 > From: Brenden Matthews <[email protected]> > Date: Mon, 13 May 2013 16:08:13 -0700 > Subject: [PATCH 10/24] Ignore unhealthy TaskTrackers. > > Review: https://reviews.apache.org/r/11117 > --- > hadoop/mesos/src/java/org/apache/hadoop/mapred/MesosScheduler.java | 4 > ++++ > 1 file changed, 4 insertions(+) > > > Diffs > ----- > > hadoop/mesos/src/java/org/apache/hadoop/mapred/MesosScheduler.java > afe401f5265e3d9494af7eace42eec45943184a3 > > Diff: https://reviews.apache.org/r/11117/diff/ > > > Testing > ------- > > Used in production at airbnb. > > > Thanks, > > Brenden Matthews > >
