ramitkataria commented on PR #64984: URL: https://github.com/apache/airflow/pull/64984#issuecomment-4226944083
For context, the reason I previously added the context workaround to add "simple context" was because the triggerrer was not ready for fetching context. Now that #55068 is merged, we can handle context using the same approach that the triggerrer does for AsyncCallbacks and the way it works in the executor for SyncCallbacks. So I think the way I would want to do implement this would involve first reverting #55241 and making changes in triggerer and executor to support context in non-task workloads like callbacks. With the current approach, we are not using the `Context` used by the rest of the codebase so it will not have support for certain fields and everything is being stored in the DB which will cause increased storage usage and slower speed compared to how it is currently handled for task. -- 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]
