xinbinhuang commented on a change in pull request #15194:
URL: https://github.com/apache/airflow/pull/15194#discussion_r611234529



##########
File path: airflow/models/variable.py
##########
@@ -47,12 +47,14 @@ class Variable(Base, LoggingMixin):
     id = Column(Integer, primary_key=True)
     key = Column(String(ID_LEN), unique=True)
     _val = Column('val', Text)
+    description = Column(Text(5000))

Review comment:
       Hmm, this is because there are other migration changes merged to the 
master before you, which happens sometimes given the number of PRs we have. We 
used to have problems with multiple diverge alembic heads, and we mitigate it 
by making sure that any PR affecting the `migration` folder will need to rebase 
on the latest master. So, to fix it, I normally delete the revision file and 
regenerate it after rebase to `master`




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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to