----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33823/#review85759 -----------------------------------------------------------
include/mesos/executor/executor.proto <https://reviews.apache.org/r/33823/#comment137514> s/an/a , also can we try to be consistent with either 'Update' or update through all the comments when refering to the tasks status enum? include/mesos/executor/executor.proto <https://reviews.apache.org/r/33823/#comment137515> see comment above include/mesos/executor/executor.proto <https://reviews.apache.org/r/33823/#comment137516> see comment above, should we add '(seee TaskState in mesos.proto)' in the first comment we refer to it? include/mesos/executor/executor.proto <https://reviews.apache.org/r/33823/#comment137525> add a comment that points to the 'Events' section above - Isabel Jimenez On May 21, 2015, 1:11 p.m., Alexander Rojas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/33823/ > ----------------------------------------------------------- > > (Updated May 21, 2015, 1:11 p.m.) > > > Review request for mesos, Isabel Jimenez, Marco Massenzio, and Vinod Kone. > > > Repository: mesos > > > Description > ------- > > See summary. > > > Diffs > ----- > > include/mesos/executor/executor.hpp PRE-CREATION > include/mesos/executor/executor.proto PRE-CREATION > src/Makefile.am 34755cf795391c9b8051a5e4acc6caf844984496 > > Diff: https://reviews.apache.org/r/33823/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Alexander Rojas > >
