RehanAhmad25 commented on PR #74539: URL: https://github.com/apache/airflow/pull/74539#issuecomment-6148367729
I checked the other DB providers. The only one that selects a client library through an extra is MySQL (`client`). JDBC uses `driver_path` and `driver_class`, and ODBC’s driver is the ODBC `driver` name, so I kept `dbapi_driver` to avoid mixing them up. If you’d prefer `client` for consistency with MySQL, I can rename it. On the docs, other providers lead with a URI example and show JSON with extras nested under `extra`, so I’ll add a URI example next to the JSON one. I’ll also switch the missing-package error to `AirflowOptionalProviderFeatureException`, like the other hooks. -- 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]
