vincbeck commented on code in PR #73506:
URL: https://github.com/apache/airflow/pull/73506#discussion_r4076631863
##########
providers/duckdb/README.rst:
##########
@@ -57,28 +60,5 @@ PIP package Version required
``duckdb`` ``>=1.2.0``
======================================= ==================
-Cross provider package dependencies
------------------------------------
-
-Those are dependencies that might be needed in order to use all the features
of the package.
-You need to install the specified providers in order to use them.
-
-You can install such cross-provider dependencies when installing from PyPI.
For example:
-
-.. code-block:: bash
-
- pip install apache-airflow-providers-duckdb[common.sql]
-
-
-============================================================================================================
==============
-Dependent package
Extra
-============================================================================================================
==============
-`apache-airflow-providers-common-sql
<https://airflow.apache.org/docs/apache-airflow-providers-common-sql>`_
``common.sql``
-============================================================================================================
==============
-
-DuckDB version compatibility
Review Comment:
I dont think there is any issue with the PR, as far as I see it, duckdb will
be included in this wave by just looking at the PR. That's only @potiuk who
mentioned it will not be included
--
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]
