----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40125/#review105797 -----------------------------------------------------------
Ship it! Ship It! - Joris Van Remoortere On Nov. 10, 2015, 5:46 a.m., Anand Mazumdar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/40125/ > ----------------------------------------------------------- > > (Updated Nov. 10, 2015, 5:46 a.m.) > > > Review request for mesos, Joris Van Remoortere and Neil Conway. > > > Repository: mesos > > > Description > ------- > > As part of MESOS-3480, we introduced an output operator for the `Executor` > struct on agent. It seems like the changes for MESOS-3772 got interpolated > with that change later leading to erroneous quotes in executor log messages. > > > Diffs > ----- > > src/slave/slave.cpp 535adc3b17d5af3fe811a8e2505f126a28212dbf > > Diff: https://reviews.apache.org/r/40125/diff/ > > > Testing > ------- > > make check. I still wonder though how this escaped without merge conflicts > when committing MESOS-3772. > > > Thanks, > > Anand Mazumdar > >