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

2015-12-15 Thread Han Zhou
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 o

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

2015-12-15 Thread Han Zhou
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 o

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

2015-12-15 Thread Ben Pfaff
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 =

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

2015-12-16 Thread Andy Zhou
On Tue, Dec 15, 2015 at 8:24 PM, Ben Pfaff 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

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

2015-12-17 Thread Andy Zhou
On Tue, Dec 15, 2015 at 7:06 PM, 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 == "upda

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

2015-12-17 Thread Han Zhou
Hi Andy, Thanks for your review. On Thu, Dec 17, 2015 at 2:00 PM, Andy Zhou wrote: > > > On Tue, Dec 15, 2015 at 7:06 PM, Han Zhou wrote: > >> Cached json objects were reused when sending notifications to >> clients. This created a problem when there were different versions >> of monitors coex

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

2015-12-17 Thread Han Zhou
On Thu, Dec 17, 2015 at 4:33 PM, Han Zhou wrote: > Hi Andy, > > Thanks for your review. > > > On Thu, Dec 17, 2015 at 2:00 PM, Andy Zhou wrote: > >> >> >> On Tue, Dec 15, 2015 at 7:06 PM, Han Zhou wrote: >> >>> Cached json objects were reused when sending notifications to >>> clients. This crea