uranusjr commented on code in PR #39032:
URL: https://github.com/apache/airflow/pull/39032#discussion_r1588662196
##########
airflow/models/dag.py:
##########
@@ -3619,7 +3619,7 @@ class DagModel(Base):
# associated zip.
fileloc = Column(String(2000))
# The base directory used by Dag Processor that parsed this dag.
- processor_subdir = Column(String(2000), nullable=True)
+ processor_subdir = Column(String(2000).with_variant(Text(length=2000),
"mysql"), nullable=True)
Review Comment:
Yeah especially other columns are just unconditionally varchar. This was
added relatively recently in #35956 so maybe @tirkarthi has reasons.
It is not a bad idea to avoid a large varchar in MySQL in general, but this
is inconsistent.
--
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]
