----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33963/#review82957 -----------------------------------------------------------
Patch looks great! Reviews applied: [31502, 31503, 31504, 33963] All tests passed. - Mesos ReviewBot On May 8, 2015, 12:14 a.m., Cong Wang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/33963/ > ----------------------------------------------------------- > > (Updated May 8, 2015, 12:14 a.m.) > > > Review request for mesos, Chi Zhang, Ian Downes, and Jie Yu. > > > Bugs: MESOS-2422 > https://issues.apache.org/jira/browse/MESOS-2422 > > > Repository: mesos > > > Description > ------- > > After adding basic filter, ARP filter implementation now can move onto basic > filter, this also matches kernel implementation too. > > > Diffs > ----- > > src/linux/routing/filter/arp.cpp 8c4976620ec296b44cefec375b5f21986cc886ae > src/linux/routing/filter/basic.hpp PRE-CREATION > src/linux/routing/filter/basic.cpp PRE-CREATION > > Diff: https://reviews.apache.org/r/33963/diff/ > > > Testing > ------- > > Run ARP filter test cases > > > Thanks, > > Cong Wang > >
