safaehar commented on code in PR #75216:
URL: https://github.com/apache/airflow/pull/75216#discussion_r4414999917
##########
airflow-core/src/airflow/api_fastapi/core_api/routes/public/pools.py:
##########
@@ -119,10 +140,22 @@ def get_pools(
session=session,
)
- pools = session.scalars(pools_select)
+ pools = session.scalars(pools_select).all()
+ if not pools:
+ return PoolCollectionResponse(pools=[], total_entries=total_entries)
+
+ pool_stats = Pool.slots_stats(
Review Comment:
i updated the aggregation to join taskInstance with pool and group by the
canonical pool.pool value. I added the analytics → Analytics mysql regression
test and verified it against a freshly initialized mysql database; both the
single and list endpoints return the expected counts now
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
