You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/02/15 21:32:21 UTC

[GitHub] [airflow] kaxil commented on a change in pull request #21273: Add test to run DB downgrade in the CI

kaxil commented on a change in pull request #21273:
URL: https://github.com/apache/airflow/pull/21273#discussion_r807332894



##########
File path: airflow/migrations/versions/5e3ec427fdd3_increase_length_of_email_and_username.py
##########
@@ -46,9 +46,11 @@ def upgrade():
 
 def downgrade():
     """Revert length of email from 256 to 64 characters"""

Review comment:
       Yeah, a feature/code in the old version (that added that revision) might be depending on the "length" at that time.
   
   So I think we should revert the chars length




-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org