ferruzzi commented on issue #14396: URL: https://github.com/apache/airflow/issues/14396#issuecomment-965594782
@potiuk Extracting a Context class out into a new module was my initial thought, but it looked like you were arguing pretty hard _for_ the TypedDict solution. A new `airflow.models.context` module won't be a trivial change and I want to make sure we're "all" on the same page before wasting a ton of time on something that will get shot down based on the implementation. Looking at the TaskInstance module, there are a few challenges with this solution. The module itself is a bit of a mess right now (I'm getting over 75 IDE warnings/errors as it stands), Context is pretty heavily entwined in it, and there would maybe/likely be a circular dependency between the TaskInstance, Context, and BaseOperator modules to sort out. This could turn into a pretty major surgery, so I'd feel better about taking it on after we have a bit of consensus that it's the path we want to go down. -- 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]
