jscheffl commented on PR #35868: URL: https://github.com/apache/airflow/pull/35868#issuecomment-1846192314
> I'm not confortable having this in 2.8.0. Let's leave it for 2.9.0 because the change is big and close to the release time. Also, I prefer we don't remove the mssql codes but instead let users know by warning that this is no longer supported if they are using mssql. I understand @potiuk concerns about removing it this way but I feel this approach is harsh. We can start by warning users and then total removal. My opinion though. I understand the change is large, this is a reason. Raised this a bit late, unfortunately review was stale for a number of days :-( Anyway, admins should have been warned already, there was a post in the docs already, see: https://airflow.apache.org/docs/apache-airflow/stable/howto/set-up-database.html#setting-up-a-mssql-database Anyway... so let put this PR on pause and merge it for 2.9.0... -- 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
