ephraimbuddy commented on a change in pull request #21273:
URL: https://github.com/apache/airflow/pull/21273#discussion_r807277045
##########
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:
Agree with you. MSSQL enforces this while other DBs don't enforce it.
I'll remove it and fix every other places we increased lengths of fields.
@ashb @jedcunningham @uranusjr Do you have contrary opinion on 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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
