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, there are two ways of doing this:
- manually change the `down_revision` number to the latest revision number
in `master` branch
- delete the revision file and regenerate it after rebase to `master` (I
personally do this.)
--
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]
