create a view for listing tournaments

alembic/versions/141ff488c041_tournament_view_generation.py

@@ -0,0 +1,64 @@
+"""view generation
+
+Revision ID: 141ff488c041
+Revises: 1891eeeaa700
+Create Date: 2025-04-20 18:53:07.272272
+
+"""
+from typing import Sequence, Union
+
+from alembic import op
+import sqlalchemy as sa
+
+
+# revision identifiers, used by Alembic.
+revision: str = '141ff488c041'
+down_revision: Union[str, None] = '1891eeeaa700'
+branch_labels: Union[str, Sequence[str], None] = None
+depends_on: Union[str, Sequence[str], None] = None
+
+
+def upgrade() -> None:
+    """Upgrade schema."""
+    sql_view = """
+        CREATE MATERIALIZED VIEW IF NOT EXISTS data.tournaments_list_m_view
+        TABLESPACE pg_default
+        AS
+        SELECT tournament_name AS name,
+            min(date_part('year'::text, date)) AS first_year,
+            max(date_part('year'::text, date)) AS last_year
+        FROM data.match
+        WHERE tournament_name IS NOT NULL
+        GROUP BY tournament_name
+        ORDER BY (lower(tournament_name::text))
+        WITH DATA;
+
+        ALTER TABLE IF EXISTS data.tournaments_list_m_view
+            OWNER TO tennis_admin;
+    """
+
+    sql_index = """
+        CREATE INDEX IF NOT EXISTS idx_tournaments_list_m_view_name
+            ON data.tournaments_list_m_view USING btree
+            (name COLLATE pg_catalog."default" ASC NULLS LAST)
+            WITH (deduplicate_items=True)
+            TABLESPACE pg_default;
+    """
+    op.execute(sa.text(sql_view))
+    op.execute(sa.text(sql_index))
+    # ### end Alembic commands ###
+
+
+def downgrade() -> None:
+    """Downgrade schema."""
+    
+    sql_view = """
+        DROP MATERIALIZED VIEW IF EXISTS data.tournaments_list_m_view;
+    """
+
+    sql_index = """
+        DROP INDEX IF EXISTS data.idx_tournaments_list_m_view_name;
+    """
+    op.execute(sa.text(sql_view))
+    op.execute(sa.text(sql_index))
+    # ### end Alembic commands ###

src/main.py

@@ -219,29 +219,18 @@ async def list_tournament_names(
     """
     List all the tournament names and first and last year of occurrence
     """
-    # Get all the tournament names
-    # and first and last year of occurrence
-    tournaments = session.query(Match.tournament_name).distinct().all()
-
-    output = []
-    # Get the first and last year of occurrence
-    for tournament in sorted([tournament[0] for tournament in tournaments], key=str.lower):
-        first_year = session.query(Match.date).filter(Match.tournament_name == tournament).order_by(Match.date.asc()).first()
-        last_year = session.query(Match.date).filter(Match.tournament_name == tournament).order_by(Match.date.desc()).first()
-        if first_year and last_year:
-            output.append({
-                "name": tournament,
-                "first_year": first_year[0].year,
-                "last_year": last_year[0].year
-            })
-        else:
-            output.append({
-                "name": tournament,
-                "first_year": None,
-                "last_year": None
-            })
-    
-    return output
+    tournaments = session.execute(text("SELECT t.name, t.first_year, t.last_year FROM data.tournaments_list_m_view AS t")).all()
+
+    tournaments = [
+        {
+            "name": tournament[0],
+            "first_year": tournament[1],
+            "last_year": tournament[2]
+        }
+        for tournament in tournaments
+    ]
+
+    return tournaments
 
 # Get a player
 @app.get("/player/{player_id}", tags=["player"], description="Get a player from the database", response_model=PlayerApiDetail)