[ovs-dev] [PATCH] ovsdb: separate json cache for different monitor versions

Andy Zhou azhou at ovn.org
Wed Dec 16 21:58:48 UTC 2015


On Tue, Dec 15, 2015 at 8:24 PM, Ben Pfaff <blp at 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 <zhouhan at 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.



More information about the dev mailing list