"dev" <dev-boun...@openvswitch.org> wrote on 07/16/2016 10:47:14 PM:

> From: Ryan Moats/Omaha/IBM@IBMUS
> To: Hui Kang <hkang.sun...@gmail.com>
> Cc: dev@openvswitch.org
> Date: 07/16/2016 10:47 PM
> Subject: Re: [ovs-dev] [PATCH v2] Scanning only changed entries in the
ovnsb
> Sent by: "dev" <dev-boun...@openvswitch.org>
>
> "dev" <dev-boun...@openvswitch.org> wrote on 07/14/2016 05:59:29 PM:
>
> > From: Hui Kang <hkang.sun...@gmail.com>
> > To: dev@openvswitch.org
> > Date: 07/14/2016 06:04 PM
> > Subject: [ovs-dev] [PATCH v2] Scanning only changed entries in the
ovnsb
> > Sent by: "dev" <dev-boun...@openvswitch.org>
> >
> > Improve performance by scanning only changed entries in ovnsb
> >
> > Signed-off-by: Hui Kang <ka...@us.ibm.com>
> > ---
>
> It applies cleanly, it compiles, it passes unit tests and
> it looks straightforward enough.  The only nit I'd pick is
> the commit description is a bit misleading - I think saying
>
> "Improve performance by scanning only changed port binding
> entries when determining whether to mark the logical switch
> port up or down"
>
> would be more accurate.  On the assumption that the merge gets
> a more descriptive commit message...

Thanks, Ryan. I have updated the commit message in v3 in this trivial fix.

- Hui

>
> Acked-by: Ryan Moats <rmo...@us.ibm.com>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to