You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2023/01/11 20:18:13 UTC

[GitHub] [airflow] mhenc commented on issue #28269: AIP-44 Migrate DagFileProcessor.execute_callbacks to Internal API.

mhenc commented on issue #28269:
URL: https://github.com/apache/airflow/issues/28269#issuecomment-1379434344

   Right, I should have been more descriptive:
   The problem about this task is that executing the callback itself must be still done in the DagProcessor (not Internal API) component.
   One the other hand all requests to DB must be done through Internal API. E.g. `_execute_task_callbacks`  first read data from DB then call `ti.handle_failure`(which unfortunatelly also tasks session in the arguments and ameks some additional DB calls.).
   This all must be untangled - even with refactoring of some methods/introducing new


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