----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32506/#review79376 -----------------------------------------------------------
Ship it! include/mesos/type_utils.hpp <https://reviews.apache.org/r/32506/#comment128651> Shall we include <ostream> as well? include/mesos/type_utils.hpp <https://reviews.apache.org/r/32506/#comment128655> `scheduler::Call_Type_Name()` for clarity? include/mesos/type_utils.hpp <https://reviews.apache.org/r/32506/#comment128656> `scheduler::Event_Type_Name()` for clarity? - Alexander Rukletsov On April 3, 2015, 11:42 p.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/32506/ > ----------------------------------------------------------- > > (Updated April 3, 2015, 11:42 p.m.) > > > Review request for mesos and Ben Mahler. > > > Bugs: MESOS-1127 > https://issues.apache.org/jira/browse/MESOS-1127 > > > Repository: mesos > > > Description > ------- > > For readable output in logs. > > > Diffs > ----- > > include/mesos/type_utils.hpp cdf5864389a72002b538c263d70bcade2bdffa45 > src/master/master.cpp 618db68ee4163b06e479cf3413eda4b63c9c5a4b > > Diff: https://reviews.apache.org/r/32506/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Vinod Kone > >