ashb edited a comment on issue #3796: [AIRFLOW-2824] - Add config to disable 
default conn creation
URL: 
https://github.com/apache/incubator-airflow/pull/3796#issuecomment-417844535
 
 
   The classes are always available (even when the rbac is not enabled) so we 
should be able to create migrations for them somehow.
   
   Edit: thinking about it a bit more I'm not sure it makes sense to create 
Alembic migrations for the Flask RBAC models: if we import the model classes in 
the migration then the migrations can change over time (we got stung with this 
already once on our _own_ migrations), or we have to duplicate the model 
classes in the migration file, which means we have to manually keep them in 
sync/notice changes.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to