----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39873/#review105191 -----------------------------------------------------------
src/master/master.cpp (lines 6004 - 6008) <https://reviews.apache.org/r/39873/#comment163614> Does this comment belong here? It seems redundant with the comments in the protobuf files. Should we just point the reader there instead if you think this warrants a comment? Seems like the reader will already refer to the 'latest_update' field definition to read more. Also would love to avoid wrapping comments at 80 when they are paragraphs (if you compare the paragraph added here vs. the one above 'if (update.has_uuid())', the latter seems easier on the eyes when I'm reading the code outside of reviewboard). - Ben Mahler On Nov. 2, 2015, 10:34 p.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39873/ > ----------------------------------------------------------- > > (Updated Nov. 2, 2015, 10:34 p.m.) > > > Review request for mesos and Ben Mahler. > > > Bugs: MESOS-2864 > https://issues.apache.org/jira/browse/MESOS-2864 > > > Repository: mesos > > > Description > ------- > > The master now doesn't change the latest state of a task if it has already > terminated. But it still updates the status update state and uuid. > > > Diffs > ----- > > src/master/master.cpp 2bc5a97a5b50c8a8a9902c47b2e9e3b5216d97ea > > Diff: https://reviews.apache.org/r/39873/diff/ > > > Testing > ------- > > make check > > Ran the "RecoverCompletedExecutor" test 100 times as it was failing most of > the time without this fix. > > > Thanks, > > Vinod Kone > >
