On Tue, Dec 15, 2015 at 8:24 PM, Ben Pfaff <b...@ovn.org> wrote:

> On Tue, Dec 15, 2015 at 07:06:34PM -0800, Han Zhou wrote:
> > Cached json objects were reused when sending notifications to
> > clients. This created a problem when there were different versions
> > of monitors coexiting. E.g. clients expecting version2 notification
> > would receive messages with method == "update2" but payload in
> > version1 format, which end up failure of processing the updates.
> >
> > This patch fixes the issue by using dedicated cache for each version.
> >
> > Signed-off-by: Han Zhou <zhou...@gmail.com>
>
> Looks good at a quick skim.
>
> Andy, will you review this?
>
Sure,  Han, thanks for the bug report and fix. I will take a closer look at
the patch.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to