jscheffl commented on code in PR #45264:
URL: https://github.com/apache/airflow/pull/45264#discussion_r1900108216
##########
task_sdk/src/airflow/sdk/definitions/taskgroup.py:
##########
@@ -270,7 +272,7 @@ def group_id(self) -> str | None:
@property
def label(self) -> str | None:
"""group_id excluding parent's group_id used as the node label in
UI."""
- return self._group_id
+ return self.group_display_name or self._group_id
Review Comment:
@uranusjr I remember you had in your mind when we created the display name
for DAG and Task that we consolidate on node class level / base class. Is it
something with TaskSDK that we should follow-up on?
--
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]
