----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47660/#review136505 -----------------------------------------------------------
src/slave/slave.cpp (lines 3943 - 3950) <https://reviews.apache.org/r/47660/#comment201560> I don't understand this change. Can you describe the motivation in the description of this rb? For instance, what if both task.command().has_user() and frameworkInfo.has_user() are false? - Jie Yu On May 26, 2016, 8:55 p.m., Gilbert Song wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47660/ > ----------------------------------------------------------- > > (Updated May 26, 2016, 8:55 p.m.) > > > Review request for mesos, Artem Harutyunyan, Jie Yu, James Peach, Kevin > Klues, and Timothy Chen. > > > Repository: mesos > > > Description > ------- > > Fixed slave switch user logic in 'getExecutorInfo'. > > > Diffs > ----- > > src/slave/slave.cpp 470b5c82ea6ff01d799b06245609725853300ef1 > > Diff: https://reviews.apache.org/r/47660/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Gilbert Song > >
