eladkal commented on code in PR #25802: URL: https://github.com/apache/airflow/pull/25802#discussion_r953051685
########## RELEASE_NOTES.rst: ########## @@ -502,6 +502,12 @@ Airflow 2.3.0 restores the original meaning to the parameter. If you set it to " can be embedded in an iframe (no header is added), but when you set it to "false" the header is added and Airflow cannot be embedded in an iframe. +``MySQLToS3Operator`` argument `pd_csv_kwargs` has beed replaced with `pd_kwargs` (#20807) +"""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""" + +Although the old one (``pd_csv_kwargs``) is still accepted but no longer taken into account. Considering +``MySQLToS3Operator`` deprecation it is recommeded to switch to ``SqlToS3Operator``. Doing this keep in mind +that ``index`` and ``header`` arguments should be moved into ``pd_kwargs`` dict. Review Comment: noting also that your assumption is incorrect. https://airflow.apache.org/docs/apache-airflow/stable/installation/installing-from-pypi.html You can install Airflow with providers and you can install providers as standalone. It's your choice if you want to couple it or not. Airflow core by itself does not bring provider dependencies. Quoting from the docs: `There is also a constraints-no-providers constraint file, which contains just constraints required to install Airflow core. This allows to install and upgrade airflow separately and independently from providers.` -- 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]
