You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "moiseenkov (via GitHub)" <gi...@apache.org> on 2023/04/06 09:00:15 UTC
[GitHub] [airflow] moiseenkov commented on pull request #30487: Remove mysql-connector-python
moiseenkov commented on PR #30487:
URL: https://github.com/apache/airflow/pull/30487#issuecomment-1498721447
Thank you all, I misunderstood the point.
I will leave the code that uses `mysql-connector-python`, but is it true that we have to exclude this package from dependencies anyway? If I get it right, leaving `mysql-connector-python` here will cause dependency conflicts.
My suggestion is removing the `mysql-connector-python` from the dependencies, and throwing an exception with a readable message whenever the package is used. Those users, who will face this issue, are always able to install this package manually and resolve possible conflicts on their own, or transition to the recommended library. @eladkal , @potiuk WDYT?
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org