FIX: move index creation to a PostgreSQL-specific trigger

Author: genematx

tiled/catalog/orm.py

@@ -109,14 +109,6 @@ class Node(Timestamped, Base):
             "access_blob",
             postgresql_using="gin",
         ),
-        # PostgreSQL-specific covering index for fast key lookups.
-        Index(
-            "ix_nodes_parent_time_id_key",
-            "parent",
-            "time_created",
-            "id",
-            postgresql_include=["key"],
-        ),
     )
 
 
@@ -299,9 +291,9 @@ def unique_parameter_num_null_check(target, connection, **kw):
 
 @event.listens_for(Node.__table__, "after_create")
 def create_index_metadata_tsvector_search(target, connection, **kw):
-    # This creates a ts_vector based metadata search index for fulltext.
-    # Postgres only feature
+    # Postgres only features
     if connection.engine.dialect.name == "postgresql":
+        # This creates a ts_vector based metadata search index for fulltext
         connection.execute(
             text(
                 """
@@ -311,6 +303,16 @@ def create_index_metadata_tsvector_search(target, connection, **kw):
                 """
             )
         )
+        # Covering B-tree index on (parent, time_created, id) INCLUDE (key)
+        connection.execute(
+            text(
+                """
+                CREATE INDEX ix_nodes_parent_time_id_key
+                ON nodes (parent, time_created, id)
+                INCLUDE (key)
+                """
+            )
+        )
 
 
 @event.listens_for(NodesClosure.__table__, "after_create")