potiuk commented on code in PR #37376: URL: https://github.com/apache/airflow/pull/37376#discussion_r1487726741
########## docs/apache-airflow-providers-common-sql/index.rst: ########## @@ -18,6 +18,20 @@ ``apache-airflow-providers-common-sql`` ======================================= +Supported Database Types +------------------------ + +The ``apache-airflow-providers-common-sql`` provider is compatible with the following +database types: + +- :doc:`PostgreSQL <apache-airflow-providers-postgres:index>` Review Comment: Yes. Agree with @Taragolis - I think it makes sense to Add it as a separate page, then in the future it will be easier to automate it, because eventually we want this page to be generated automatically. This should be as easy as adding a new .rst file and add it to the Index (above the "the remainder of this page is generated automatically). One more thing - I think it should be a much more complete list as well: From the slack thread, this the complete list: Generated with: ``` find . -name index.rst | grep -v '_api' | xargs grep -l common.sql ``` ``` /docs/apache-airflow-providers-postgres/index.rst ./docs/apache-airflow-providers-apache-impala/index.rst ./docs/apache-airflow-providers-elasticsearch/index.rst ./docs/apache-airflow-providers-microsoft-mssql/index.rst ./docs/apache-airflow-providers-slack/index.rst ./docs/apache-airflow-providers-databricks/index.rst ./docs/apache-airflow-providers-vertica/index.rst ./docs/apache-airflow-providers-jdbc/index.rst ./docs/apache-airflow-providers-apache-hive/index.rst ./docs/apache-airflow-providers-apache-druid/index.rst ./docs/apache-airflow-providers-mysql/index.rst ./docs/apache-airflow-providers-odbc/index.rst ./docs/apache-airflow-providers-snowflake/index.rst ./docs/apache-airflow-providers-amazon/index.rst ./docs/apache-airflow-providers-common-sql/index.rst ./docs/apache-airflow-providers-presto/index.rst ./docs/apache-airflow-providers-trino/index.rst ./docs/apache-airflow-providers-apache-pinot/index.rst ./docs/apache-airflow-providers-pgvector/index.rst ./docs/apache-airflow-providers-sqlite/index.rst ./docs/apache-airflow-providers-exasol/index.rst ./docs/apache-airflow-providers-google/index.rst ./docs/apache-airflow-providers-openlineage/index.rst ./docs/apache-airflow-providers-apache-drill/index.rst ``` -- 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]
