rawwar opened a new issue, #39586:
URL: https://github.com/apache/airflow/issues/39586

   ### Description
   
   Currently, we allow users to pass dialect via `sqlalchemy_scheme` extra 
field of the `mssql` connections. I am proposing to remove this field as it is 
not possible to support multiple dialects without installing the relevant 
dialect driver. Also, the [documentation 
page](https://airflow.apache.org/docs/apache-airflow-providers-microsoft-mssql/stable/connections/mssql.html)
 does not even mention about `sqlalchemy_scheme` or the possibility of using 
other dialects.
   
   ### Use case/motivation
   
   Supporting multiple dialects requires the installation of multiple drivers. 
Since we only install `pymssql`, this allows the usage of the `mssql+pymssql` 
dialect alone and not others.
   
   I am also interested in updating the documentation to reflect this.
   
   ### Related issues
   
   _No response_
   
   ### Are you willing to submit a PR?
   
   - [X] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of 
Conduct](https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md)
   


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