fix: Don't copy ancestor's meta, to avoid indicating incorrect counts

open-contracting · May 25, 2024 · 8aa6d59 · 8aa6d59
1 parent 7296179
commit 8aa6d59
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/workers/extract/dataset_filter.py b/workers/extract/dataset_filter.py
@@ -79,7 +79,7 @@ def callback(client_state, channel, method, properties, input_message):
         cursor.execute(
             """\
             INSERT INTO dataset (name, meta, ancestor_id)
-            SELECT name, meta, NULL FROM dataset WHERE id = %(dataset_id)s
+            SELECT name, '{}'::jsonb, NULL FROM dataset WHERE id = %(dataset_id)s
             RETURNING id
             """,
             {"dataset_id": dataset_id_original},

diff --git a/workers/extract/kingfisher_process.py b/workers/extract/kingfisher_process.py
@@ -58,10 +58,10 @@ def callback(client_state, channel, method, properties, input_message):
             cursor.execute(
                 """\
                 INSERT INTO dataset (name, meta, ancestor_id)
-                VALUES (%(name)s, %(meta)s, %(ancestor_id)s)
+                VALUES (%(name)s, '{}'::jsonb, %(ancestor_id)s)
                 RETURNING id
                 """,
-                {"name": name, "meta": Json({}), "ancestor_id": ancestor_id},
+                {"name": name, "ancestor_id": ancestor_id},
             )
             dataset_id = cursor.fetchone()[0]