diff mbox series

[ovs-dev,2/3] ovn-northd-ddlog: Remove deltas from output tables when we use them.

Message ID 20210310002440.3449903-2-blp@ovn.org
State Accepted
Headers show
Series [ovs-dev,1/3] ovn-northd-ddlog: Fix memory leaks. | expand

Commit Message

Ben Pfaff March 10, 2021, 12:24 a.m. UTC 
Otherwise we will see these deltas every time we go through the main
loop, which is harmless but unnecessary.

Signed-off-by: Ben Pfaff <blp@ovn.org>
---
 northd/ovn-northd-ddlog.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
diff mbox series

Patch

diff --git a/northd/ovn-northd-ddlog.c b/northd/ovn-northd-ddlog.c
index be3de5cee29e..80d3aa8d8aeb 100644
--- a/northd/ovn-northd-ddlog.c
+++ b/northd/ovn-northd-ddlog.c
@@ -575,7 +575,7 @@  northd_update_probe_interval(struct northd_ctx *nb, struct northd_ctx *sb)
      * database. */
     int probe_interval = 0;
     table_id tid = ddlog_get_table_id("Northd_Probe_Interval");
-    ddlog_delta *probe_delta = ddlog_delta_get_table(delta, tid);
+    ddlog_delta *probe_delta = ddlog_delta_remove_table(delta, tid);
     ddlog_delta_enumerate(probe_delta, northd_update_probe_interval_cb, (uintptr_t) &probe_interval);
     ddlog_free_delta(probe_delta);
 
@@ -961,7 +961,7 @@  get_database_ops(struct northd_ctx *ctx)
     int64_t new_sb_cfg = old_sb_cfg;
     if (ctx->has_timestamp_columns) {
         table_id sb_cfg_tid = ddlog_get_table_id("SbCfg");
-        ddlog_delta *sb_cfg_delta = ddlog_delta_get_table(delta, sb_cfg_tid);
+        ddlog_delta *sb_cfg_delta = ddlog_delta_remove_table(delta, sb_cfg_tid);
         ddlog_delta_enumerate(sb_cfg_delta, northd_update_sb_cfg_cb,
                               (uintptr_t) &new_sb_cfg);
         ddlog_free_delta(sb_cfg_delta);