> On March 8, 2017, 1:11 a.m., Kevin Klues wrote: > > This patch both does what the summary says AND adds the > > DELETE_NESTED_CONTAINER API call. Should we separate these into two > > separate patches?
Yes, they belong in https://reviews.apache.org/r/57387/, thanks! - Gastón ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57341/#review168228 ----------------------------------------------------------- On March 8, 2017, 11:11 a.m., Gastón Kleiman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57341/ > ----------------------------------------------------------- > > (Updated March 8, 2017, 11:11 a.m.) > > > Review request for mesos, Alexander Rukletsov and Vinod Kone. > > > Bugs: MESOS-7120 > https://issues.apache.org/jira/browse/MESOS-7120 > > > Repository: mesos > > > Description > ------- > > Followed the convention in 'master/http.cpp' and explicitly used the > 'mesos::' prefix. > > > Diffs > ----- > > src/slave/http.cpp bc8209cb81194ebc8b604c9ba0d4a9e176cff2f6 > > > Diff: https://reviews.apache.org/r/57341/diff/2/ > > > Testing > ------- > > > Thanks, > > Gastón Kleiman > >