jedcunningham commented on PR #29714:
URL: https://github.com/apache/airflow/pull/29714#issuecomment-1444766801

   > The problem is not on applying migrations, but loading the migrations into 
memory without applying, which is needed for the check option.
   
   Could we use 
[REVISION_HEAD_MAP](https://github.com/apache/airflow/blob/671b88eb3423e86bb331eaf7829659080cbd184e/airflow/utils/db.py#L54)
 and `airflow.version` to short circuit in released versions of Airflow, 
leaving the expensive loading to dev envs?
   
   > I am concerned about prematurely handling it as we did with expose_config
   
   I think using a string from the start would mean we don't need the clunky 
backcompat code. Only an issue if the bool makes it to a release, but still, it 
probably makes sense to plan ahead in case we don't get to it before the next 
release.


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