jscheffl commented on PR #64068: URL: https://github.com/apache/airflow/pull/64068#issuecomment-4106904109
> @jscheffl Why not push XCom’s directly on task_instance of trigger? Why? Because now the XCom’s are passed via the EventTrigger which is fully persisted in DB. I would push them directly via task_instance on trigger, it won’t be persisted with the event in the DB, which can grow easily with XCom’s. Does not work, the `self.task_instance` in the triggerer execution is from type `airflow.executors.workloads.TaskInstance` and this is a minimal stup with no `xcom_push()` implementation. Also a manual XCom push is not working as in the triggerer code there is no DB session. So, yes this proposal here has the trade-off that XCom is serialized once in the event and then another time from the event back to XCom. -- 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]
