ephraimbuddy commented on PR #41437: URL: https://github.com/apache/airflow/pull/41437#issuecomment-2298690646
> > These changes are part of AIP-79, do you think we still need another mailing thread on it? > > To clarify, I am not saying no to the what (the merit of AIP-79) I am saying no to the how. > > Let me rephrase - for Airflow 2 users who runs fab provider in any future version (with this PR included) will there be migration scripts running as part of the provider? If the answer is yes then I have a concern here. There is no migration script currently. However, if anyone has migration to add, they can add it on the FAB provider side. What this PR intends to achieve is for users to add their migrations on the FAB side and have a way for airflow to run such migrations. Even before this PR, a pre-commit forbids adding FAB migration to the core. -- 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]
