The GitHub Actions job "Tests" on 
airflow.git/handle-parsing-result-owns-session has succeeded.
Run started by GitHub user ephraimbuddy (triggered by ephraimbuddy).

Head commit for run:
ccd0bd83452746505629636145158c0869812f8e / Ephraim Anierobi 
<[email protected]>
Decouple session ownership and drop unused bundle_version from parse-result 
helpers

The projected AIP-92 DB-to-API swap needs DagFileProcessorManager subclasses
that forward parser results through an API instead of the metadata DB.
``handle_parsing_result`` is already public and the obvious override point,
but its caller ``_collect_results`` is wrapped in ``@provide_session`` and
threads the session in -- so an override that does not need a session has
to either accept and ignore one or replace ``_collect_results`` itself just
to dodge the decorator.

Move the ``@provide_session`` decorator down onto ``handle_parsing_result``
and stop passing the session from ``_collect_results``. The default
behaviour, the persistence path, and the per-cycle error isolation are
unchanged; subclasses overriding ``handle_parsing_result`` can now run
without a session being created on their behalf.

Also drop the unused ``bundle_version`` parameter from ``process_parse_results``
(and its caller / tests) -- it was threaded through but never read.

Report URL: https://github.com/apache/airflow/actions/runs/25121505406

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to