----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11484/#review21509 -----------------------------------------------------------
Ship it! hadoop/mesos/src/java/org/apache/hadoop/mapred/MesosScheduler.java <https://reviews.apache.org/r/11484/#comment44549> How about changing this to LOG.debug(). That way, if there is something you want to debug, we can turn up the log level. Also, as benm mentioned, change this to "Removing completed job..." - Vinod Kone On June 3, 2013, 5:59 p.m., Brenden Matthews wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11484/ > ----------------------------------------------------------- > > (Updated June 3, 2013, 5:59 p.m.) > > > Review request for mesos. > > > Description > ------- > > Don't log every single completed task. > > On large jobs (with many thousands of tasks) this can generate an > unreasonable number of log messages. > > > Diffs > ----- > > hadoop/mesos/src/java/org/apache/hadoop/mapred/MesosScheduler.java > afe401f5265e3d9494af7eace42eec45943184a3 > > Diff: https://reviews.apache.org/r/11484/diff/ > > > Testing > ------- > > > Thanks, > > Brenden Matthews > >
