From cbf39fe7e874855c82aa7d2c77eb648aa8fbe0c4 Mon Sep 17 00:00:00 2001 From: xiaohuanlin <33860497+xiaohuanlin@users.noreply.github.com> Date: Mon, 28 Jul 2025 05:07:51 -0400 Subject: [PATCH] [v3-0-test] Remove unnecessary group_by clause in event logs query (#53733) The group_by(Log.id) clause was redundant since Log.id is the primary key and doesn't affect the query results. This simplifies the query without changing functionality. Fixes #53695 (cherry picked from commit 85d80cdecd25ce3d6d57e36a71825b8af32218fa) Co-authored-by: xiaohuanlin <33860497+xiaohuanlin@users.noreply.github.com> --- .../airflow/api_fastapi/core_api/routes/public/event_logs.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/airflow-core/src/airflow/api_fastapi/core_api/routes/public/event_logs.py b/airflow-core/src/airflow/api_fastapi/core_api/routes/public/event_logs.py index 33b3c68e564a7..e721162389991 100644 --- a/airflow-core/src/airflow/api_fastapi/core_api/routes/public/event_logs.py +++ b/airflow-core/src/airflow/api_fastapi/core_api/routes/public/event_logs.py @@ -116,7 +116,7 @@ def get_event_logs( ], ) -> EventLogCollectionResponse: """Get all Event Logs.""" - query = select(Log).group_by(Log.id) + query = select(Log) event_logs_select, total_entries = paginated_select( statement=query, order_by=order_by,