It looks like SQLite doesn't actually enforce the max length on the columns:
```
sqlite> insert into connection (password)
values('aaaaaaaaaaaaaaaaaaaaaaaaa...a');
sqlite> select id, length(password) from connection;
1|1
2|1900
```
So maybe the fix is to simply not run this migration on sqlite engines? I'm
some what surprised that the tests we added in #3516 didn't catch this (as the
model already has the length set at 5000 but we were missing a migration to
change it. (Which is nothing to do with this PR).
[ Full content available at:
https://github.com/apache/incubator-airflow/pull/3851 ]
This message was relayed via gitbox.apache.org for [email protected]