mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jie Yu" <yujie....@gmail.com>
Subject Re: Review Request 34957: Fixed a bug in qdisc search function.
Date Tue, 02 Jun 2015 23:11:31 GMT


> On June 2, 2015, 11:09 p.m., Paul Brett wrote:
> > src/linux/routing/queueing/internal.hpp, line 143
> > <https://reviews.apache.org/r/34957/diff/1/?file=976870#file976870line143>
> >
> >     Not your problem, but we really should either explain why we have to increment
the reference counter here but nowhere else or just fix Netlink to hide this (it does hide
most of the cleanup logic)

Yeah, we should definitely add some comments. Can you sugguest how to fix Netlink to hide
this?


- Jie


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/34957/#review86303
-----------------------------------------------------------


On June 2, 2015, 10:05 p.m., Jie Yu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/34957/
> -----------------------------------------------------------
> 
> (Updated June 2, 2015, 10:05 p.m.)
> 
> 
> Review request for mesos, Chi Zhang, Ian Downes, Paul Brett, and Cong Wang.
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Fixed a bug in qdisc search function.
> 
> 
> Diffs
> -----
> 
>   src/linux/routing/queueing/internal.hpp d43a9fd405af2e59dc57cfc7ba9b5e77cb9f6b70 
> 
> Diff: https://reviews.apache.org/r/34957/diff/
> 
> 
> Testing
> -------
> 
> sudo make check
> 
> 
> Thanks,
> 
> Jie Yu
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message