----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31901/#review75923 -----------------------------------------------------------
Ship it! Master (48dc4c7) is green with this patch. ./build-support/jenkins/build.sh I will refresh this build result if you post a review containing "@ReviewBot retry" - Aurora ReviewBot On March 10, 2015, 6 p.m., Bill Farner wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31901/ > ----------------------------------------------------------- > > (Updated March 10, 2015, 6 p.m.) > > > Review request for Aurora and Maxim Khutornenko. > > > Repository: aurora > > > Description > ------- > > I realized that the reason values apply to more than just TASK_LOST, so the > previous code would hide reasons like when a task has exceeded its memory > limit. > > > Diffs > ----- > > src/main/java/org/apache/aurora/scheduler/mesos/TaskStatusStats.java > 68214f20dd7f46adec2d8f6d84e9840dc88dc0fb > src/test/java/org/apache/aurora/scheduler/mesos/TaskStatusStatsTest.java > 4bbeff957050fb9d8ee81d9fc79520a6a0ac38a1 > > Diff: https://reviews.apache.org/r/31901/diff/ > > > Testing > ------- > > > Thanks, > > Bill Farner > >