----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66462/#review200957 -----------------------------------------------------------
Are all of these necessary for now? src/slave/slave.cpp Lines 8034-8038 (patched) <https://reviews.apache.org/r/66462/#comment281872> Move these outside the block. - Chun-Hung Hsiao On April 11, 2018, 5:58 p.m., Gaston Kleiman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/66462/ > ----------------------------------------------------------- > > (Updated April 11, 2018, 5:58 p.m.) > > > Review request for mesos and Greg Mann. > > > Repository: mesos > > > Description > ------- > > Added new operation states to be used for status reconciliation. > > > Diffs > ----- > > include/mesos/mesos.proto 676f0b090cad7ebf59eb32556f17ff8b5f247907 > include/mesos/v1/mesos.proto 10d506517c9f098374ab0c8f4dcfda42e1896c95 > src/common/protobuf_utils.cpp 141a444534b776a2c90e2a0daf9727cd21e39080 > src/master/master.cpp f7da675e8fe96159e5335c9e83b65b67ed90eda8 > src/slave/slave.cpp e5d6c3fac5054a6b0a0b5b77abd850a35be6ccc5 > > > Diff: https://reviews.apache.org/r/66462/diff/3/ > > > Testing > ------- > > `sudo bin/mesos-tests` on GNU/Linux > > > Thanks, > > Gaston Kleiman > >
