jason810496 commented on code in PR #67235: URL: https://github.com/apache/airflow/pull/67235#discussion_r3274194034
########## task-sdk/src/airflow/sdk/execution_time/schema/versions/__init__.py: ########## Review Comment: > especially since there are additional supervisor-only models that may change as well but aren’t tracked in the Execution API spec Exactly, this is what the migrator and the versions bundle here trying to resolve. Yes, there're partial of data models are directly from Execution API spec. but most of them aren't define in Execution API, like all the data models defined in `task-sdk/src/airflow/sdk/execution_time/comms.py`. From my perspective, no matter the data models define in Execution API or not, all the data models consumed by `CommsDecoder` should be versioned somewhere. -- 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]
