seruman commented on issue #65010:
URL: https://github.com/apache/airflow/issues/65010#issuecomment-4237795522

   > I think this diff _might_ fix it.
   > 
   > ```diff
   > diff --git 
i/airflow-core/src/airflow/api_fastapi/core_api/routes/ui/grid.py 
w/airflow-core/src/airflow/api_fastapi/core_api/routes/ui/grid.py
   > index 156982d409..48f4559bf8 100644
   > --- i/airflow-core/src/airflow/api_fastapi/core_api/routes/ui/grid.py
   > +++ w/airflow-core/src/airflow/api_fastapi/core_api/routes/ui/grid.py
   > @@ -27,7 +27,7 @@ from sqlalchemy import exists, select
   >  from sqlalchemy.orm import Session, joinedload, load_only
   >  
   >  from airflow.api_fastapi.auth.managers.models.resource_details import 
DagAccessEntity
   > -from airflow.api_fastapi.common.db.common import SessionDep, 
paginated_select
   > +from airflow.api_fastapi.common.db.common import SessionDep, 
paginated_select, _get_session
   >  from airflow.api_fastapi.common.db.dag_runs import 
attach_dag_versions_to_runs
   >  from airflow.api_fastapi.common.parameters import (
   >      QueryDagRunRunTypesFilter,
   > @@ -453,7 +453,8 @@ def _build_ti_summaries(
   >  )
   >  def get_grid_ti_summaries_stream(
   >      dag_id: str,
   > -    session: SessionDep,
   > +    # session: SessionDep,
   > +    session: Annotated[Session, Depends(_get_session)],
   >      run_ids: Annotated[list[str] | None, Query()] = None,
   >  ) -> StreamingResponse:
   >      """
   > 
   > ```
   
   I was about the try something similar and ended up in the same path where 
clean up somehow was not getting called and connection would left open. I'm 
trying to setup a minimal reproducer. Thanks @ashb 
   
   Issue is also observable in MySQL. 


-- 
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]

Reply via email to