[GitHub] [airflow] mhenc commented on pull request #29516: AIP-44 Migrate BaseJob.run to Internal API

["mhenc (via GitHub)" <gi...@apache.org>]

mhenc commented on PR #29516:
URL: https://github.com/apache/airflow/pull/29516#issuecomment-1429518246

   I don't think this is what we need to do there (sorry I was not more explicit in the bug description).
   
   the `self.execute()` needs to be run locally (e.g on worker) as this is the part that touches customer code, your change will make it executed in Internal API sever.
   
   What we need to do there is migrate everything that touches "session" - so the pre-execute (changing state etc) and post-execute - updateing state,end_date.


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