Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Docs][Connector][Source][doc]add Parallel parallelism #5310

Merged
merged 3 commits into from
Sep 11, 2023
Merged

[Docs][Connector][Source][doc]add Parallel parallelism #5310

merged 3 commits into from
Sep 11, 2023

Conversation

zhilinli123
Copy link
Contributor

Purpose of this pull request

Check list

liugddx
liugddx reviewed Aug 21, 2023
View reviewed changes
docs/en/connector-v2/source/Mysql.md
@@ -114,9 +114,13 @@ sink {
> Read your query table in parallel with the shard field you configured and the shard data You can do this if you want to read the whole table

```
env {
execution.parallelism = 10
Copy link
Member

Choose a reason for hiding this comment

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

Modifying this requires specifying partition_column otherwise it has no effect.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

Copy link
Contributor Author

Choose a reason for hiding this comment

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

image
Because it's hidden you can't see it when you unfold it

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Uploading image.png…

liugddx
liugddx approved these changes Aug 22, 2023
View reviewed changes
Copy link
Member

@liugddx liugddx left a comment

Choose a reason for hiding this comment

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

LGTM

@zhilinli123
Copy link
Contributor Author

@Hisoka-X @EricJoy2048 PTAL thks

@EricJoy2048 EricJoy2048 merged commit cde42a3 into apache:dev Sep 11, 2023
32 checks passed
Zhouwen-CN pushed a commit to Zhouwen-CN/seatunnel that referenced this pull request Sep 11, 2023
@zhilinli123 @Zhouwen-CN
[Docs][Connector][Source][doc]add Parallel parallelism (apache#5310)
9bbe581
Zhouwen-CN pushed a commit to Zhouwen-CN/seatunnel that referenced this pull request Sep 11, 2023
@zhilinli123 @Zhouwen-CN
[Docs][Connector][Source][doc]add Parallel parallelism (apache#5310)
b4dbbfe
gnehil pushed a commit to gnehil/seatunnel that referenced this pull request Oct 12, 2023
@zhilinli123
[Docs][Connector][Source][doc]add Parallel parallelism (apache#5310)
8c5953f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@EricJoy2048 EricJoy2048 EricJoy2048 approved these changes

@liugddx liugddx liugddx approved these changes

Assignees
No one assigned
Labels
approved reviewed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@zhilinli123 @EricJoy2048 @liugddx