----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34832/#review86061 -----------------------------------------------------------
src/tests/routing_tests.cpp <https://reviews.apache.org/r/34832/#comment137947> Please add full-stop punctuation to comments. src/tests/routing_tests.cpp <https://reviews.apache.org/r/34832/#comment137953> Not yours, but why ASSERT? if this fails then the later expections should fail (rather than cause assertions)? src/tests/routing_tests.cpp <https://reviews.apache.org/r/34832/#comment137943> Is ASSERT needed: if it's not some/true then will it trigger other uncontrolled assertions? src/tests/routing_tests.cpp <https://reviews.apache.org/r/34832/#comment137948> ditto here and other comments. src/tests/routing_tests.cpp <https://reviews.apache.org/r/34832/#comment137945> ditto src/tests/routing_tests.cpp <https://reviews.apache.org/r/34832/#comment137944> ditto src/tests/routing_tests.cpp <https://reviews.apache.org/r/34832/#comment137952> move this up to the ingress::create("noSuchInterface")? src/tests/routing_tests.cpp <https://reviews.apache.org/r/34832/#comment137949> These seem to be the sames as lines 448:450 without any obvious modication between there and here? src/tests/routing_tests.cpp <https://reviews.apache.org/r/34832/#comment137946> FqCodel src/tests/routing_tests.cpp <https://reviews.apache.org/r/34832/#comment137951> move this up to the fq_code::create("noSuchInterface")? src/tests/routing_tests.cpp <https://reviews.apache.org/r/34832/#comment137950> ditto, duplicates of lines 492:494? - Ian Downes On May 29, 2015, 3:09 p.m., Paul Brett wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34832/ > ----------------------------------------------------------- > > (Updated May 29, 2015, 3:09 p.m.) > > > Review request for mesos, Chi Zhang, Ian Downes, Jie Yu, and Cong Wang. > > > Bugs: MESOS-2781 > https://issues.apache.org/jira/browse/MESOS-2781 > > > Repository: mesos > > > Description > ------- > > Add new qdisc tests for fq_codel and ingress queueing discipline. > > > Diffs > ----- > > src/tests/routing_tests.cpp 2066a56571f43a012992b4294fbf5b6e07c2b159 > > Diff: https://reviews.apache.org/r/34832/diff/ > > > Testing > ------- > > > Thanks, > > Paul Brett > >