ephraimbuddy opened a new pull request, #66110: URL: https://github.com/apache/airflow/pull/66110
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. --- ##### Was generative AI tooling used to co-author this PR? - [x] Yes (please specify the tool below) Generated-by: Claude code (Opus 4.7) -- 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]
