> On May 6, 2013, 11:44 p.m., Ben Mahler wrote: > >
Great! I recall hearing about such a scenario being possible, glad to see a fix :) - Ben ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10920/#review20232 ----------------------------------------------------------- On May 3, 2013, 6:45 p.m., Brenden Matthews wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10920/ > ----------------------------------------------------------- > > (Updated May 3, 2013, 6:45 p.m.) > > > Review request for mesos. > > > Description > ------- > > From 9a67cc50605cce529f469c2899729c0799d70970 Mon Sep 17 00:00:00 2001 > From: Brenden Matthews <[email protected]> > Date: Fri, 26 Apr 2013 11:42:18 -0700 > Subject: [PATCH 2/9] Don't prematurely kill TaskTrackers. > > We assign the JobID to the internal Mesos tracker (for the TaskTracker) > rather than the TaskID. > > In the case where a TaskTracker has only map tasks assigned to it and > they have all completed, we mustn't terminate the tracker until the > entire job has finished, including the map tasks. > --- > .../org/apache/hadoop/mapred/MesosScheduler.java | 368 > ++++++++++---------- > 1 file changed, 188 insertions(+), 180 deletions(-) > > > Diffs > ----- > > hadoop/mesos/src/java/org/apache/hadoop/mapred/MesosScheduler.java afe401f > > Diff: https://reviews.apache.org/r/10920/diff/ > > > Testing > ------- > > Used in production at airbnb. > > > Thanks, > > Brenden Matthews > >
