Skip to content

[Feature-8024][Document] Add example and notice about task type Flink#688

Merged
zhongjiajie merged 24 commits intoapache:masterfrom
QuakeWang:flink-task
Feb 12, 2022
Merged

[Feature-8024][Document] Add example and notice about task type Flink#688
zhongjiajie merged 24 commits intoapache:masterfrom
QuakeWang:flink-task

Conversation

@QuakeWang
Copy link
Copy Markdown
Contributor

@QuakeWang QuakeWang commented Feb 11, 2022

close: apache/dolphinscheduler#8024

I have updated the document of the Flink task, please review it. If there is nothing need to modify, I will translate and update the other relevant files.

PTAL @zhongjiajie

@zhongjiajie zhongjiajie self-requested a review February 11, 2022 13:06
@zhongjiajie zhongjiajie added enhancement New feature or request task labels Feb 11, 2022
Copy link
Copy Markdown
Member

@zhongjiajie zhongjiajie left a comment

Choose a reason for hiding this comment

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

I add some suggestions, others LTGM

@zhongjiajie
Copy link
Copy Markdown
Member

Please remember to add English version and change dev version too.

@QuakeWang
Copy link
Copy Markdown
Contributor Author

Please remember to add English version and change dev version too.

I have completed it.

@zhongjiajie zhongjiajie added the waiting user response Waiting PR submiter response label Feb 12, 2022
Copy link
Copy Markdown
Member

@zhongjiajie zhongjiajie left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for your contribution

@zhongjiajie zhongjiajie removed the waiting user response Waiting PR submiter response label Feb 12, 2022
@zhongjiajie
Copy link
Copy Markdown
Member

Oh, we still need to pass CI

@zhongjiajie zhongjiajie merged commit 938c814 into apache:master Feb 12, 2022
@QuakeWang QuakeWang deleted the flink-task branch February 14, 2022 03:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request task

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature][Document] Add example and notice about task type Flink

2 participants