mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Paul Brett" <pau...@twopensource.com>
Subject Re: Review Request 34957: Fixed a bug in qdisc search function.
Date Tue, 02 Jun 2015 23:22:19 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)
> 
> Jie Yu wrote:
>     Yeah, we should definitely add some comments. Can you sugguest how to fix Netlink
to hide this?

Chi and I were talking about adding a copy constructor to Netlink that would increment the
reference counter using helper functions like the destructors, but that needs a separate ticket
and some more thought.


- Paul


-----------------------------------------------------------
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