dkranchii commented on code in PR #68403:
URL: https://github.com/apache/airflow/pull/68403#discussion_r3538960101
##########
task-sdk/src/airflow/sdk/types.py:
##########
@@ -124,6 +124,7 @@ class DagRunProtocol(Protocol):
consumed_asset_events: list[AssetEventDagRunReference]
partition_key: str | None
note: str | None
+ team_name: str | None
Review Comment:
@ashb Given the design context @o-nikolas laid out above - team lives on
the bundle and DagRun sub-model on TIRunContext is the pragmatic carrier that
already ships from the server - this PR is purely a typing sync with the
already-shipped Execution API schema. If moving team_name off DagRun
server-side is desirable, happy to file a follow-up issue. If you're OK with
landing this typing sync as-is, could you resolve the thread?
--
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]
