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]

Reply via email to