Message ID | 20201218053144.2637583-3-blp@ovn.org |
---|---|
State | Accepted |
Headers | show |
Series | Refactor OVSDB IDL into two layers | expand |
On 12/18/20 6:31 AM, Ben Pfaff wrote: > ovsdb_idl_db_parse_monitor_reply() clears the IDL and parses the > received data. There's no need to do it again afterward. > > Signed-off-by: Ben Pfaff <blp@ovn.org> > Fixes: 1b1d2e6daa56 ("ovsdb: Introduce experimental support for clustered databases.") > --- > lib/ovsdb-idl.c | 3 --- > 1 file changed, 3 deletions(-) > LGTM, Acked-by: Ilya Maximets <i.maximets@ovn.org> > diff --git a/lib/ovsdb-idl.c b/lib/ovsdb-idl.c > index efaa08a1e98f..fb638c499c51 100644 > --- a/lib/ovsdb-idl.c > +++ b/lib/ovsdb-idl.c > @@ -819,9 +819,6 @@ ovsdb_idl_process_response(struct ovsdb_idl *idl, struct jsonrpc_msg *msg) > ovsdb_idl_db_parse_monitor_reply(&idl->data, msg->result, > OVSDB_IDL_MM_MONITOR); > idl->data.change_seqno++; > - ovsdb_idl_clear(idl); > - ovsdb_idl_db_parse_update(&idl->data, msg->result, > - OVSDB_IDL_MM_MONITOR); > break; > > case IDL_S_MONITORING: >
diff --git a/lib/ovsdb-idl.c b/lib/ovsdb-idl.c index efaa08a1e98f..fb638c499c51 100644 --- a/lib/ovsdb-idl.c +++ b/lib/ovsdb-idl.c @@ -819,9 +819,6 @@ ovsdb_idl_process_response(struct ovsdb_idl *idl, struct jsonrpc_msg *msg) ovsdb_idl_db_parse_monitor_reply(&idl->data, msg->result, OVSDB_IDL_MM_MONITOR); idl->data.change_seqno++; - ovsdb_idl_clear(idl); - ovsdb_idl_db_parse_update(&idl->data, msg->result, - OVSDB_IDL_MM_MONITOR); break; case IDL_S_MONITORING:
ovsdb_idl_db_parse_monitor_reply() clears the IDL and parses the received data. There's no need to do it again afterward. Signed-off-by: Ben Pfaff <blp@ovn.org> Fixes: 1b1d2e6daa56 ("ovsdb: Introduce experimental support for clustered databases.") --- lib/ovsdb-idl.c | 3 --- 1 file changed, 3 deletions(-)