Skip to content

Conversation

Cleverdada
Copy link
Contributor

Update readme: Add links for connectors and databases, for 943

Copy link
Contributor

@leonardBang leonardBang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @Cleverdada for the contribution, left minor comments

README.md Outdated

| Protocol | Connector | Database | Driver |
|------------|----------------------------------------------------------------------------------------------------------------|------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|-------------------------|
| MySQL | [mysql-cdc](https://ververica.github.io/flink-cdc-connectors/master/content/connectors/mysql-cdc.html) | <li> [MySQL](https://dev.mysql.com/doc): 5.7, 8.0.x <li> [MariaDB](https://mariadb.org): 10.x <li> [PolarDB MySQL](https://www.aliyun.com/product/polardb): 5.7, 8.0.x <li> [PolarDB X](https://github.com/ApsaraDB/galaxysql): 2.0.1 <li> [RDS MySQL](https://www.aliyun.com/product/rds/mysql): 5.7, 8.0.x <li> [Aurora MySQL](https://aws.amazon.com/cn/rds/aurora): 5.7, 8.0.x | JDBC Driver: 8.0.16 |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MySQL 5.6 has been supported in #396, we can add it for all mysql protocol database.

README.md Outdated
| Protocol | Connector | Database | Driver |
|------------|----------------------------------------------------------------------------------------------------------------|------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|-------------------------|
| MySQL | [mysql-cdc](https://ververica.github.io/flink-cdc-connectors/master/content/connectors/mysql-cdc.html) | <li> [MySQL](https://dev.mysql.com/doc): 5.7, 8.0.x <li> [MariaDB](https://mariadb.org): 10.x <li> [PolarDB MySQL](https://www.aliyun.com/product/polardb): 5.7, 8.0.x <li> [PolarDB X](https://github.com/ApsaraDB/galaxysql): 2.0.1 <li> [RDS MySQL](https://www.aliyun.com/product/rds/mysql): 5.7, 8.0.x <li> [Aurora MySQL](https://aws.amazon.com/cn/rds/aurora): 5.7, 8.0.x | JDBC Driver: 8.0.16 |
| PostgreSQL | [postgresql-cdc](https://ververica.github.io/flink-cdc-connectors/master/content/connectors/postgres-cdc.html) | <li> [PostgreSQL](https://www.postgresql.org): 9.6, 10, 11, 12 | JDBC Driver: 42.2.12 |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

when we cut branch release-2.2 from master, we don't want to change the link, could you use [postgresql-cdc](postgres-cdc.md) ?

README.md Outdated
| Sqlserver | Database: 2017, 2019 <br/>JDBC Driver: 7.2.2.jre8|
## Supported (Tested) Databases

| Protocol | Connector | Database | Driver |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We shouldn't use protocol word here after think twice. We always say one database vendor is compatible with MySQL but don’t say it follows MySQL protocol, and other databases are not protocols.

Copy link
Contributor

@leonardBang leonardBang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@leonardBang leonardBang merged commit 5ab7131 into apache:master Mar 16, 2022
ChaomingZhangCN pushed a commit to ChaomingZhangCN/flink-cdc that referenced this pull request Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants