> On June 1, 2015, 5:59 p.m., Chi Zhang wrote: > > src/linux/routing/queueing/fq_codel.cpp, line 102 > > <https://reviews.apache.org/r/34830/diff/2/?file=975051#file975051line102> > > > > ignore if you have done in a different patch: > > > > egress::ROOT? (You had ingress::ROOT) > > Chi Zhang wrote: > feel free to drop this and do it in a different patch though
Ideally yes, but it is probaly not worth doing until we need more in the egress namespace than just this one symbol. > On June 1, 2015, 5:59 p.m., Chi Zhang wrote: > > src/linux/routing/queueing/internal.hpp, lines 128-131 > > <https://reviews.apache.org/r/34830/diff/2/?file=975054#file975054line128> > > > > reinterpret_cast is the preferred c++ way, but I am not sure if the > > codebase has done similiar casting with this before?. > > > > If using this introdue consistency, use another patch to do it all? > > Jie Yu wrote: > +1 reinterpret_case is used in stout and twice in mesos (in jvm.hpp and common/thread.cpp). Converting all the C style casts to C++ would be a significant effort. - Paul ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34830/#review86006 ----------------------------------------------------------- On May 30, 2015, 9 p.m., Paul Brett wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34830/ > ----------------------------------------------------------- > > (Updated May 30, 2015, 9 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 > ------- > > Fix routing ingress/fq_codel search returning wrong qdisc. > > Add new checking to verify that operations occur on the correct interface and > change the behaviour of create() operation to return false rather than an > error if creation fails if the interface does not exist (now consistent with > exists() and remove()). > > > Diffs > ----- > > src/linux/routing/queueing/fq_codel.hpp > 7de1e31d4c43ac9cbffab1e472ea51140719f900 > src/linux/routing/queueing/fq_codel.cpp > 64b5c73e8cfa672141ddb663e879c58b4babfdbc > src/linux/routing/queueing/ingress.hpp > 84506fecd01522471a7998176c28bea8f1367aed > src/linux/routing/queueing/ingress.cpp > ae0c38d2215e7fabcc1060e7385484bd455e1699 > src/linux/routing/queueing/internal.hpp > d43a9fd405af2e59dc57cfc7ba9b5e77cb9f6b70 > > Diff: https://reviews.apache.org/r/34830/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Paul Brett > >
