rkhachatryan commented on a change in pull request #16898:
URL: https://github.com/apache/flink/pull/16898#discussion_r694604553
##########
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 think for production, users would still go to the vendor website to
check the driver version and distribution they should use (regardless whether
first-time users or not); or use some previously used driver.
One issue with these links is that we have to rely on users feedback to see
if these link are still correct.
They are also a bit inconsistent: for Postgres it's Postgres website and for
MySQL it's maven repo.
So IMO we shouldn't keep them.
--
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]
