----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53624/#review155657 -----------------------------------------------------------
src/cli/execute.cpp (line 299) <https://reviews.apache.org/r/53624/#comment225674> thinking a bit more, this flag should be "content_type" with "protobuf" as default. that way if we support more content types in the future, this would be future proof. - Vinod Kone On Nov. 10, 2016, 5:46 p.m., James Peach wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53624/ > ----------------------------------------------------------- > > (Updated Nov. 10, 2016, 5:46 p.m.) > > > Review request for mesos, Anand Mazumdar, Jie Yu, and Vinod Kone. > > > Bugs: MESOS-6562 > https://issues.apache.org/jira/browse/MESOS-6562 > > > Repository: mesos > > > Description > ------- > > Use JSON content type in mesos-execute so that packet tracing > protocol interactions is easier. > > > Diffs > ----- > > src/cli/execute.cpp b47c427c5ad29dda1985ee8fef6c4efe054df879 > > Diff: https://reviews.apache.org/r/53624/diff/ > > > Testing > ------- > > make check. Manually verified with tcpdump. > > > Thanks, > > James Peach > >