Skip to content

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

Merged
EricJoy2048 merged 3 commits into
apache:devfrom
zhilinli123:feature-doc-parallelism
Sep 11, 2023
Merged

[Docs][Connector][Source][doc]add Parallel parallelism#5310
EricJoy2048 merged 3 commits into
apache:devfrom
zhilinli123:feature-doc-parallelism

Conversation

@zhilinli123
Copy link
Copy Markdown
Contributor

Purpose of this pull request

Check list


```
env {
execution.parallelism = 10
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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…

Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

@Hisoka-X @EricJoy2048 PTAL thks

@EricJoy2048 EricJoy2048 merged commit cde42a3 into apache:dev Sep 11, 2023
Zhouwen-CN pushed a commit to Zhouwen-CN/seatunnel that referenced this pull request Sep 11, 2023
Zhouwen-CN pushed a commit to Zhouwen-CN/seatunnel that referenced this pull request Sep 11, 2023
gnehil pushed a commit to gnehil/seatunnel that referenced this pull request Oct 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants