ephraimbuddy commented on PR #35868:
URL: https://github.com/apache/airflow/pull/35868#issuecomment-1844942742

   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.


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

Reply via email to