> On June 2, 2015, 10:40 p.m., Paul Brett wrote: > > src/linux/routing/queueing/discipline.hpp, line 39 > > <https://reviews.apache.org/r/34956/diff/1/?file=976864#file976864line39> > > > > Should config be optional? > > Jie Yu wrote: > We still need a Config struct for each type of qdisc so that the template > specification could work. It just looks weired to me to do > ``` > Discipline<ingress::Config>( > ... > None()); > ``` > > Do you have a strong opinion on that?
Synced with Paul offline. Decided to punt this for now. I'll add a TODO. - Jie ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34956/#review86300 ----------------------------------------------------------- On June 2, 2015, 10:04 p.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34956/ > ----------------------------------------------------------- > > (Updated June 2, 2015, 10:04 p.m.) > > > Review request for mesos, Chi Zhang, Ian Downes, Paul Brett, and Cong Wang. > > > Repository: mesos > > > Description > ------- > > Refactored the queueing discipline data structure. > > This is for us to more easily extend qdisc support. > > > Diffs > ----- > > src/Makefile.am f2110a360b6e2a1e99e1d9630d5dfacfd818530a > src/linux/routing/queueing/discipline.hpp PRE-CREATION > src/linux/routing/queueing/fq_codel.hpp > 7de1e31d4c43ac9cbffab1e472ea51140719f900 > src/linux/routing/queueing/fq_codel.cpp > 4dc2a9d2ed52937f0a78a083980db488c06b45a7 > 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/34956/diff/ > > > Testing > ------- > > sudo make check > > > Thanks, > > Jie Yu > >