tatiana commented on PR #29714: URL: https://github.com/apache/airflow/pull/29714#issuecomment-1441949861
> I have wondered if we should have three values for this. true (default) migrates the database automatically, check only checks if the database is up-to-date, and false does nothing (this PR). But I’m not sure how expensive loading the Alembic migration graph is; it might be too expensive for this to make sense. That's an exciting thought, @uranusjr! Are there any existing Airflow benchmarks related to applying migrations? I need more information to make a call on this - it would be great if we could have some data to make this decision. Do we have any other configurations which have these three states? If we move this way, we will change the config from boolean to string - should we use an enum or some other data structure to represent them..? Would we want to handle "1", "true", "True" and the other variations as well? -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
