jedcunningham opened a new pull request, #55231:
URL: https://github.com/apache/airflow/pull/55231

   We don't "always" need the FAB provider, we just need it if we don't have 
the db at the version we expect. This can be because 1) we upgraded from AF2 
and the tables still exist or 2) the instance had the FAB provider in the past 
but doesn't any longer. In either case though, we have the tables, and at the 
version we expect anyway, so no need to require the FAB provider to be 
installed before the downgrade.
   
   I tested the following manually:
   
   - with fab provider, with "known" db version
   - with fab provider, with newer db version
   - with fab provider, with no db version 
   - without fab provider, with "known" db version (worked! this is the point 
of the PR)
   - without fab provider, with newer db version (requires fab provider to be 
installed)
   - without fab provider, without fab db (requires fab provider to be 
installed)


-- 
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]

Reply via email to