Skip to content

Conversation

@scienceyang
Copy link
Contributor

Purpose of this pull request

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

@davidzollo davidzollo added the First-time contributor First-time contributor label Mar 5, 2025
@@ -0,0 +1,342 @@
# MySQL CDC

> MySQL CDC 源连接器
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
> MySQL CDC 源连接器
> MySQL CDC Source 连接器

Copy link
Contributor

@davidzollo davidzollo left a comment

Choose a reason for hiding this comment

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

Please update the PR

| log_bin | ON |
+--------------------------+----------------+
5 rows in set (0.00 sec)
```

This comment was marked as outdated.

davidzollo

This comment was marked as outdated.

davidzollo

This comment was marked as outdated.

davidzollo
davidzollo previously approved these changes Apr 14, 2025
Copy link
Contributor

@davidzollo davidzollo left a comment

Choose a reason for hiding this comment

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

+1
LGTM

Comment on lines 321 to 325
### 表名支持正则以读取多个表
> `table-pattern` 和 `table-names` 只能选择一个
```hocon
env {
# You can set engine configuration here
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
### 表名支持正则以读取多个表
> `table-pattern``table-names` 只能选择一个
```hocon
env {
# You can set engine configuration here
### 表名支持正则以读取多个表
> `table-pattern``table-names` 只能选择一个
```hocon
env {
# You can set engine configuration here

Copy link
Contributor

@joyCurry30 joyCurry30 left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution. I left some comment. Please take a look. @scienceyang


## 描述

MySQL CDC连接器允许从MySQL数据库读取快照和增量数据. 本文档描述了如何设置MySQL CDC连接器以针对MySQL数据库运行SQL查询.
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like so weird.

Comment on lines 41 to 42
> 1. You need to ensure that the [jdbc 驱动 jar package](https://mvnrepository.com/artifact/mysql/mysql-connector-java) 已经放在目录 `${SEATUNNEL_HOME}/lib/`.

Copy link
Contributor

Choose a reason for hiding this comment

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

Why not translate this?


### 启用MySQL Binlog

一定要为MySQL复制启用二进制日志。二进制日志记录事务更新以供复制工具传播更改.
Copy link
Contributor

Choose a reason for hiding this comment

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

"二进制日志" -> binlog.


#### 配置MySQL session超时时长

当为量大的数据库初始一致快照时,在读取表的过程中,您已建立的连接可能会超时。您可以通过在MySQL配置文件中配置interactive_timeout(交互超时时间)和wait_timeout(等待超时时间)来防止这种行为.
Copy link
Contributor

Choose a reason for hiding this comment

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

What's the meaning of "当为量大的"?

@github-actions github-actions bot removed the reviewed label May 10, 2025
Copy link
Contributor

@davidzollo davidzollo left a comment

Choose a reason for hiding this comment

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

+1
Good job

@hailin0 hailin0 merged commit 9ed09c2 into apache:dev May 20, 2025
5 checks passed
joexjx pushed a commit to joexjx/seatunnel that referenced this pull request May 21, 2025
dybyte pushed a commit to dybyte/seatunnel that referenced this pull request Jul 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants