You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "bobu-putheeckal (via GitHub)" <gi...@apache.org> on 2026/05/24 13:59:09 UTC

[PR] Fix backfill API session handling for SQLite [airflow]

bobu-putheeckal opened a new pull request, #67411:
URL: https://github.com/apache/airflow/pull/67411

   ## Summary
   - pass the request-scoped API session into `_create_backfill` when creating backfills
   - keep existing CLI/model behavior by letting `_create_backfill` create its own session when one is not supplied
   - add a regression test that fails if the API path opens a separate backfill model session
   
   Fixes #66726
   
   ## Testing
   - `uv run --no-dev --package apache-airflow-core --with 'apache-airflow-devel-common[basic]' ruff check airflow-core/src/airflow/models/backfill.py airflow-core/src/airflow/api_fastapi/core_api/routes/public/backfills.py airflow-core/tests/unit/api_fastapi/core_api/routes/public/test_backfills.py`
   - `PYTHONPATH="airflow-core/src:airflow-core/tests:devel-common/src:providers/git/src:providers/standard/src:providers/fab/src:shared/configuration/src:shared/dagnode/src:shared/listeners/src:shared/logging/src:shared/module_loading/src:shared/observability/src:shared/plugins_manager/src:shared/providers_discovery/src:shared/secrets_backend/src:shared/secrets_masker/src:shared/serialization/src:shared/state/src:shared/template_rendering/src:shared/timezones/src" uv run --python 3.12 --no-dev --package apache-airflow-core --with-editable ./devel-common --with 'apache-airflow-devel-common[basic]' --with-editable ./providers/git --with-editable ./providers/standard --with-editable ./providers/fab python -m pytest airflow-core/tests/unit/api_fastapi/core_api/routes/public/test_backfills.py::TestCreateBackfill -q`
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [x] Yes — OpenAI Codex
   
   Generated-by: OpenAI Codex following the guidelines in contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix backfill API session handling for SQLite [airflow]

Posted by "kaxil (via GitHub)" <gi...@apache.org>.
kaxil commented on PR #67411:
URL: https://github.com/apache/airflow/pull/67411#issuecomment-4529871999

   PR template hasn't been followed on any of your PRs. As such I am closing all your PRs since I see little to no evidence of actual testing and mass PRs to Airflow & other repos.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix backfill API session handling for SQLite [airflow]

Posted by "kaxil (via GitHub)" <gi...@apache.org>.
kaxil closed pull request #67411: Fix backfill API session handling for SQLite
URL: https://github.com/apache/airflow/pull/67411


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org