> On Nov. 5, 2015, 1:41 a.m., Ben Mahler wrote: > > src/master/master.cpp, lines 4410-4418 > > <https://reviews.apache.org/r/39792/diff/2/?file=1114172#file1114172line4410> > > > > Would be great to continue wrapping comment paragraphs at 70 even > > though it's not a hard rule, for consistency with the large amount of > > existing comment paragraphs and to make it easier to read.
>From http://www.mail-archive.com/dev%40mesos.apache.org/msg32944.html it >looked like we agreed to wrap all new and updated comments around 80 chars? >I'll make sure they are not jagged. - Vinod ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39792/#review105200 ----------------------------------------------------------- On Nov. 2, 2015, 7:32 p.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39792/ > ----------------------------------------------------------- > > (Updated Nov. 2, 2015, 7:32 p.m.) > > > Review request for mesos, Anand Mazumdar and Ben Mahler. > > > Repository: mesos > > > Description > ------- > > This just makes sure master and slave properly set the uuid in task status to > setup the stage for deprecating the messy logic in scheduler driver in a > future release. > > > Diffs > ----- > > src/master/master.hpp b76d30197b3decda0a742e03ce01a17a64b633ac > src/master/master.cpp 9f4586e668a2141f4937497d42853fbdea7751a5 > src/sched/sched.cpp 9c5e3b8e42605f4647d897ba02ea3a17e494f355 > src/slave/slave.cpp 5f9b52b41eaab0c24965f28e192074340e00bde5 > > Diff: https://reviews.apache.org/r/39792/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Vinod Kone > >
