kaxil commented on issue #14459: URL: https://github.com/apache/airflow/issues/14459#issuecomment-797782559
>@kaxil please correct me if any point is incorrect (as per our discussion at #14581 (comment)) or in case you have any other things to add. Thanks. Yup that is correct, if we rename or remove the DagModel it will break the migrations. Historically we have created a Dummy Class instead of importing one from Airflow to avoid such situation, example: https://github.com/apache/airflow/blob/4372d456154a6922e0c0547a487af3cdadb43b4a/airflow/migrations/versions/8646922c8a04_change_default_pool_slots_to_1.py#L48-L72 ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
