----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28616/#review63621 -----------------------------------------------------------
Ship it! Thanks Jie! src/master/master.cpp <https://reviews.apache.org/r/28616/#comment105904> Is this TODO still needed? Looks like we'll send back the message now, which I assume will reference the name of the invalid resource? - Ben Mahler On Dec. 3, 2014, 12:15 a.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28616/ > ----------------------------------------------------------- > > (Updated Dec. 3, 2014, 12:15 a.m.) > > > Review request for mesos and Ben Mahler. > > > Bugs: MESOS-2167 > https://issues.apache.org/jira/browse/MESOS-2167 > > > Repository: mesos-git > > > Description > ------- > > Removed the empty resource check in master. > > > Diffs > ----- > > src/master/master.cpp 8fcda4b9b5857f14cff8f6af2de31cca0208b88d > src/tests/resource_offers_tests.cpp > 7e09319727992505b0b98af0943f9541c5d182e2 > > Diff: https://reviews.apache.org/r/28616/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Jie Yu > >
