-
Notifications
You must be signed in to change notification settings - Fork 13.8k
[FLINK-33041][docs] Add an article to guide users to migrate their DataSet jobs to DataStream #23362
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.
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
Conversation
4de1347
to
a381381
Compare
@flinkbot run azure |
a381381
to
f107b31
Compare
docs/content/docs/dev/datastream/how_to_migrate_from_dataset_to_datastream.md
Outdated
Show resolved
Hide resolved
docs/content/docs/dev/datastream/how_to_migrate_from_dataset_to_datastream.md
Outdated
Show resolved
Hide resolved
docs/content/docs/dev/datastream/how_to_migrate_from_dataset_to_datastream.md
Outdated
Show resolved
Hide resolved
docs/content/docs/dev/datastream/how_to_migrate_from_dataset_to_datastream.md
Outdated
Show resolved
Hide resolved
docs/content/docs/dev/datastream/how_to_migrate_from_dataset_to_datastream.md
Outdated
Show resolved
Hide resolved
docs/content/docs/dev/datastream/how_to_migrate_from_dataset_to_datastream.md
Outdated
Show resolved
Hide resolved
docs/content/docs/dev/datastream/how_to_migrate_from_dataset_to_datastream.md
Outdated
Show resolved
Hide resolved
docs/content/docs/dev/datastream/how_to_migrate_from_dataset_to_datastream.md
Outdated
Show resolved
Hide resolved
docs/content/docs/dev/datastream/how_to_migrate_from_dataset_to_datastream.md
Outdated
Show resolved
Hide resolved
docs/content/docs/dev/datastream/how_to_migrate_from_dataset_to_datastream.md
Outdated
Show resolved
Hide resolved
f107b31
to
64ffb74
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR. Left some comments as below.
64ffb74
to
3595dac
Compare
Thanks for the careful review of @yunfengzhou-hub . I've updated a round of changes. Please take a look when you have time. 😊 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the update! Left some comments as below.
3595dac
to
5b67c57
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the update. Looks much better now. Left some comments as below.
@xintongsong Could you please take a look at this PR?
5b67c57
to
02efa99
Compare
Thanks for the update! LGTM overall except the idea to divide transformations into 4 categories. |
d61a401
to
321203c
Compare
321203c
to
2677c87
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@WencongLiu, thanks for addressing my comments. LGTM.
According to the contribution guidelines, please add an identical markdown file in content.zh/
and open a Jira ticket for the translation task.
…to migrate their DataSet jobs to DataStream
…to migrate their DataSet jobs to DataStream
2677c87
to
4884314
Compare
Thanks for the review! I have added an identical markdown file in |
…to migrate their DataSet jobs to DataStream This closes #23362
What is the purpose of the change
Add an article to guide users to migrate their DataSet jobs to DataStream.
Brief change log
Does this pull request potentially affect one of the following parts:
@Public(Evolving)
: (no)Documentation