"dev" <[email protected]> wrote on 07/16/2016 10:47:14 PM:

> From: Ryan Moats/Omaha/IBM@IBMUS
> To: Hui Kang <[email protected]>
> Cc: [email protected]
> Date: 07/16/2016 10:47 PM
> Subject: Re: [ovs-dev] [PATCH v2] Scanning only changed entries in the
ovnsb
> Sent by: "dev" <[email protected]>
>
> "dev" <[email protected]> wrote on 07/14/2016 05:59:29 PM:
>
> > From: Hui Kang <[email protected]>
> > To: [email protected]
> > Date: 07/14/2016 06:04 PM
> > Subject: [ovs-dev] [PATCH v2] Scanning only changed entries in the
ovnsb
> > Sent by: "dev" <[email protected]>
> >
> > Improve performance by scanning only changed entries in ovnsb
> >
> > Signed-off-by: Hui Kang <[email protected]>
> > ---
>
> 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 <[email protected]>
> _______________________________________________
> dev mailing list
> [email protected]
> http://openvswitch.org/mailman/listinfo/dev
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to