curcur commented on a change in pull request #16898:
URL: https://github.com/apache/flink/pull/16898#discussion_r695568262
##########
File path: docs/content/docs/connectors/datastream/jdbc.md
##########
@@ -32,14 +32,6 @@ To use it, add the following dependency to your project
(along with your JDBC dr
{{< artifact flink-connector-jdbc withScalaVersion >}}
-A driver dependency is also required to connect to a specified database. Here
are drivers currently supported:
-
-| Driver | Group Id | Artifact Id | JAR
|
-| :-----------| :------------------| :----------------------|
:----------------|
-| MySQL | `mysql` | `mysql-connector-java` |
[Download](https://repo.maven.apache.org/maven2/mysql/mysql-connector-java/) |
-| PostgreSQL | `org.postgresql` | `postgresql` |
[Download](https://jdbc.postgresql.org/download.html) |
-| Derby | `org.apache.derby` | `derby` |
[Download](http://db.apache.org/derby/derby_downloads.html) |
-
Review comment:
I do not have a strong opinion on this.
- Those things (listed) do not change very often.
- The point here is to give some guidances. It will provide some hints for
users and for other not listed dbs what they are looking for.
- JDBC SQL Connector provides similar guides:
https://ci.apache.org/projects/flink/flink-docs-release-1.13/docs/connectors/table/jdbc/
But again, up to you! I am fine to remove it as well.
--
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]
