Skip relations without a member for monitor

Author: Rub21

images/tiler-monitor/pipeline-monitor/checks/imposm_import.py

@@ -218,6 +218,8 @@ def _get_changeset_elements(changeset_id):
                 timestamp = elem.attrib.get("timestamp", "")
                 # Count nodes for ways (to detect invalid geometries)
                 node_count = len(elem.findall("nd")) if elem_type == "way" else 0
+                # Count members for relations (to detect empty relations)
+                member_count = len(elem.findall("member")) if elem_type == "relation" else 0
                 elements.append({
                     "type": elem_type,
                     "osm_id": int(osm_id),
@@ -226,6 +228,7 @@ def _get_changeset_elements(changeset_id):
                     "tags": tags,
                     "timestamp": timestamp,
                     "node_count": node_count,
+                    "member_count": member_count,
                 })
     return elements
 

images/tiler-monitor/pipeline-monitor/imposm_config_loader.py

@@ -254,6 +254,16 @@ def get_skip_reason(elem, tag_to_check, table_details, importable_relation_types
     if not tags:
         return {"reason": "Element has no tags", "commentable": False}
 
+    # Skip relations with no members — imposm cannot build geometry from an
+    # empty relation, so it will never appear in the tiler DB.
+    if elem_type == "relation":
+        member_count = elem.get("member_count", -1)
+        if member_count == 0:
+            return {
+                "reason": "Relation has no members — imposm cannot import empty relations",
+                "commentable": False,
+            }
+
     # Check non-importable relation types early.
     # Relations with types like "waterway" are grouping relations whose member
     # ways get imported individually — the relation itself is never stored.