----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16188/#review30224 -----------------------------------------------------------
Ship it! - Brenden Matthews On Dec. 11, 2013, 8:07 p.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/16188/ > ----------------------------------------------------------- > > (Updated Dec. 11, 2013, 8:07 p.m.) > > > Review request for mesos, Benjamin Hindman and Brenden Matthews. > > > Repository: mesos-git > > > Description > ------- > > The fix in https://reviews.apache.org/r/16038/ was not quite right, it will > continually replace the task status, which fortunately only results in the > webui start / finish information being incorrect. > > This is a fix as was brought up by Brenden, to compare against the status > state instead of the task state. > > > Diffs > ----- > > src/master/master.cpp cb8e613b31f39329d61a490f5d3f74fd44ffb08c > src/slave/slave.cpp 9a8b82fdd451daae71958112c0ccbc84b55f1615 > > Diff: https://reviews.apache.org/r/16188/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Ben Mahler > >
