Skip to content

Commit

Permalink
Update JDBC connectors to reflect SQL DELETE support
Browse files Browse the repository at this point in the history
  • Loading branch information
jhlodin authored and electrum committed Jul 23, 2021
1 parent 3c8042b commit cac8762
Show file tree
Hide file tree
Showing 10 changed files with 26 additions and 9 deletions.
1 change: 1 addition & 0 deletions docs/src/main/sphinx/connector/clickhouse.rst
Expand Up @@ -135,6 +135,7 @@ Limitations

The following SQL statements aren't supported:

* :doc:`/sql/delete`
* :doc:`/sql/grant`
* :doc:`/sql/revoke`
* :doc:`/sql/show-grants`
Expand Down
5 changes: 3 additions & 2 deletions docs/src/main/sphinx/connector/memsql.rst
Expand Up @@ -85,14 +85,15 @@ The connector supports pushdown for a number of operations:
* :ref:`limit-pushdown`
* :ref:`topn-pushdown`

Limitations
SQL support
-----------

The following SQL statements are not yet supported:

* :doc:`/sql/delete`
* :doc:`/sql/grant`
* :doc:`/sql/revoke`
* :doc:`/sql/show-grants`
* :doc:`/sql/show-roles`
* :doc:`/sql/show-role-grants`

.. include:: sql-delete-limitation.fragment
5 changes: 3 additions & 2 deletions docs/src/main/sphinx/connector/mysql.rst
Expand Up @@ -132,14 +132,15 @@ The connector supports pushdown for a number of operations:
* :func:`var_pop`
* :func:`var_samp`

Limitations
SQL support
-----------

The following SQL statements are not yet supported:

* :doc:`/sql/delete`
* :doc:`/sql/grant`
* :doc:`/sql/revoke`
* :doc:`/sql/show-grants`
* :doc:`/sql/show-roles`
* :doc:`/sql/show-role-grants`

.. include:: sql-delete-limitation.fragment
5 changes: 3 additions & 2 deletions docs/src/main/sphinx/connector/oracle.rst
Expand Up @@ -368,15 +368,16 @@ The connector supports pushdown for a number of operations:
* :ref:`limit-pushdown`
* :ref:`topn-pushdown`

Limitations
SQL support
-----------

The following SQL statements are not supported:

* :doc:`/sql/delete`
* :doc:`/sql/alter-table`
* :doc:`/sql/grant`
* :doc:`/sql/revoke`
* :doc:`/sql/show-grants`
* :doc:`/sql/show-roles`
* :doc:`/sql/show-role-grants`

.. include:: sql-delete-limitation.fragment
4 changes: 4 additions & 0 deletions docs/src/main/sphinx/connector/phoenix.rst
Expand Up @@ -179,3 +179,7 @@ Property Name Default Value Description
``bloomfilter`` ``NONE`` Bloomfilter to use. Valid values are ``NONE`` (default), ``ROW``, or ``ROWCOL``.
=========================== ================ ==============================================================================================================

SQL support
-----------

.. include:: sql-delete-limitation.fragment
2 changes: 2 additions & 0 deletions docs/src/main/sphinx/connector/postgresql.rst
Expand Up @@ -133,6 +133,8 @@ statements, the connector supports the following features:
* :doc:`/sql/delete`
* :ref:`sql-schema-table-management`

.. include:: sql-delete-limitation.fragment

.. _postgresql-pushdown:

Pushdown
Expand Down
5 changes: 3 additions & 2 deletions docs/src/main/sphinx/connector/redshift.rst
Expand Up @@ -77,12 +77,13 @@ Type mapping

.. include:: jdbc-type-mapping.fragment

Limitations
SQL support
-----------

The following SQL statements are not yet supported:

* :doc:`/sql/delete`
* :doc:`/sql/grant`
* :doc:`/sql/revoke`
* :doc:`/sql/show-grants`

.. include:: sql-delete-limitation.fragment
5 changes: 5 additions & 0 deletions docs/src/main/sphinx/connector/sql-delete-limitation.fragment
@@ -0,0 +1,5 @@
SQL DELETE
^^^^^^^^^^

If a ``WHERE`` clause is specified, the ``DELETE`` operation only works if the
predicate in the clause can be fully pushed down to the data source.
1 change: 1 addition & 0 deletions docs/src/main/sphinx/connector/sqlserver.rst
Expand Up @@ -126,6 +126,7 @@ supports the following features:
* :doc:`/sql/delete`
* :ref:`sql-schema-table-management`

.. include:: sql-delete-limitation.fragment

.. _sqlserver-pushdown:

Expand Down
2 changes: 1 addition & 1 deletion docs/src/main/sphinx/sql/delete.rst
Expand Up @@ -35,4 +35,4 @@ Limitations
-----------

Some connectors have limited or no support for ``DELETE``.
See connector documentation for more details.
See connector documentation for more details.

0 comments on commit cac8762

Please sign in to comment.