shubham22 commented on code in PR #32520:
URL: https://github.com/apache/airflow/pull/32520#discussion_r1341793301


##########
airflow/models/taskinstance.py:
##########
@@ -436,6 +436,7 @@ class TaskInstance(Base, LoggingMixin):
     next_method = Column(String(1000))
     next_kwargs = Column(MutableDict.as_mutable(ExtendedJSON))
 
+    _task_display_name = Column("task_display_name", String(2000), default="")

Review Comment:
   I am slightly more inclined with @uranusjr suggestion as it'd keep the DB 
clean going forward. In any case, I agree that it is not a big deal and think 
we should move forward with this PR as it is quite valuable for the users. 
   @jens-scheffler-bosch - any major concerns from you if @xgao1023  goes with 
2nd approach? 



-- 
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]

Reply via email to