----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50516/#review144155 -----------------------------------------------------------
hmm, why can't the TaskID's have spaces? They are generated by the framework and are free to contain spaces. - Anand Mazumdar On July 27, 2016, 5:29 p.m., Neil Conway wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50516/ > ----------------------------------------------------------- > > (Updated July 27, 2016, 5:29 p.m.) > > > Review request for mesos and Anand Mazumdar. > > > Repository: mesos > > > Description > ------- > > Executor IDs should be quoted, task IDs should not be. > > > Diffs > ----- > > src/master/validation.cpp 52002beac29c371411348cb026a216e99ac96ab2 > src/slave/slave.cpp c19a63e4f8df11d8150071ef427bd8038584586b > > Diff: https://reviews.apache.org/r/50516/diff/ > > > Testing > ------- > > make check, visual inspection. > > > Thanks, > > Neil Conway > >
