----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49445/#review141066 -----------------------------------------------------------
Fix it, then Ship it! src/files/files.cpp (lines 356 - 358) <https://reviews.apache.org/r/49445/#comment206447> Do you need the `defer` here anymore? - Anand Mazumdar On July 6, 2016, 11:33 a.m., Abhishek Dasgupta wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/49445/ > ----------------------------------------------------------- > > (Updated July 6, 2016, 11:33 a.m.) > > > Review request for mesos, Anand Mazumdar, zhou xing, haosdent huang, and > Vinod Kone. > > > Bugs: MESOS-5487 > https://issues.apache.org/jira/browse/MESOS-5487 > > > Repository: mesos > > > Description > ------- > > Updated FilesProcess to support List_Files Call in Operator API v1. > > > Diffs > ----- > > src/files/files.hpp b767d5bc5bee16e3bd98199773a6bc7d30c1c32d > src/files/files.cpp a5a1b86e14f63e5e3834a2900270252fbe16f586 > src/tests/files_tests.cpp 31337e280c6224a8c949c8868a53e5a785b4573f > > Diff: https://reviews.apache.org/r/49445/diff/ > > > Testing > ------- > > > Thanks, > > Abhishek Dasgupta > >