Skip to content

DataStream-api-package-guidance#2583

Merged
leonardBang merged 2 commits intoapache:masterfrom
gtk96:DataStream_API_package_guidance
Nov 26, 2023
Merged

DataStream-api-package-guidance#2583
leonardBang merged 2 commits intoapache:masterfrom
gtk96:DataStream_API_package_guidance

Conversation

@gtk96
Copy link
Copy Markdown
Contributor

@gtk96 gtk96 commented Oct 25, 2023

closed #1729

@gtk96
Copy link
Copy Markdown
Contributor Author

gtk96 commented Oct 25, 2023

image According to the pom package, the data can be displayed normally in my cluster. The guava version conflict between flink 1.13 and mysql cdc can be effectively resolved.

@gtk96
Copy link
Copy Markdown
Contributor Author

gtk96 commented Oct 25, 2023

This can run in the cluster, but it can't debug it in IntelliJ IDEA. It will report that gauva 18 can't be found. How to solve it?

@gtk96 gtk96 force-pushed the DataStream_API_package_guidance branch from 85a9edd to 7c17421 Compare October 25, 2023 03:14
@gtk96
Copy link
Copy Markdown
Contributor Author

gtk96 commented Oct 25, 2023


## frame version

flink 1.13.6 flink mysql cdc 2.4.1
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Flink 1.13 may not be support in the future, you can give an example based on the latest support version.

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.

You're right, I plan to update it to Flink 1.18 and CDC 2.5.0 in the future, because at this time, many users who do not use Flink 1.18 will be affected by the Guava version upgrade, just like some users who currently use Flink 1.13 and CDC 2.4.1 will be affected by Guava 30 and Guava 18

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Could we list all flink version and give the example in Flink 1.17.2 which is the latest stable version Flink CDC supported ? Btw,we can use mysql cdc 2.4.2 version here.

@gtk96
Copy link
Copy Markdown
Contributor Author

gtk96 commented Oct 31, 2023

@lvyanquan Can you help me see if there is any problem with the basic pom example? I can easily modify the version.

@lvyanquan
Copy link
Copy Markdown
Contributor

This may be duplicated with this doc Usage for DataStream API. Not sure whether we should add a new page for maven package.

@gtk96
Copy link
Copy Markdown
Contributor Author

gtk96 commented Nov 3, 2023

This may be duplicated with this doc Usage for DataStream API. Not sure whether we should add a new page for maven package.

cc @leonardBang @ruanhang1993 @GOODBOY008 @Jiabao-Sun

@leonardBang
Copy link
Copy Markdown
Contributor

This may be duplicated with this doc Usage for DataStream API. Not sure whether we should add a new page for maven package.

@lvyanquan This PR aims to telling users how to package a DS project which used flink cdc connectors, like flink shaded guava and mysql driver, these dependenc,ies are different in each flink cdc version. The link you posted just a DS quickstart for each cdc connector DS API, it's not enough to build DS application using flink cdc connectors。

@leonardBang
Copy link
Copy Markdown
Contributor

@gtk96 Maye we can link the guidance to [Usage for DataStream API](https://ververica.github.io/flink-cdc-connectors/master/content/about.html#usage-for-datastream-api) section of each cdc connectors

@gtk96
Copy link
Copy Markdown
Contributor Author

gtk96 commented Nov 6, 2023

@gtk96 Maye we can link the guidance to [Usage for DataStream API](https://ververica.github.io/flink-cdc-connectors/master/content/about.html#usage-for-datastream-api) section of each cdc connectors

We have so many connectors that I can't verify them one by one.😫

Copy link
Copy Markdown
Contributor

@leonardBang leonardBang left a comment

Choose a reason for hiding this comment

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

Thanks @gtk96 for the contribution, I left two comments

Comment thread docs/content/quickstart/datastream-api-package-guidance.md Outdated
Comment thread docs/content/quickstart/datastream-api-package-guidance.md Outdated

## frame version

flink 1.13.6 flink mysql cdc 2.4.1
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Could we list all flink version and give the example in Flink 1.17.2 which is the latest stable version Flink CDC supported ? Btw,we can use mysql cdc 2.4.2 version here.

Comment thread docs/content/quickstart/datastream-api-package-guidance.md Outdated
Comment thread docs/content/快速上手/datastream-api-package-guidance-zh.md Outdated
Copy link
Copy Markdown
Contributor

@leonardBang leonardBang left a comment

Choose a reason for hiding this comment

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

Thanks @gtk96 and @lvyanquan for the update, LGTM
I will append a commit to add 1.18 support as well

@leonardBang leonardBang force-pushed the DataStream_API_package_guidance branch from 7a20661 to c7e5177 Compare November 26, 2023 05:18
@leonardBang leonardBang merged commit 0bf378d into apache:master Nov 26, 2023
@leonardBang leonardBang added this to the V3.0.0 milestone Nov 26, 2023
@gtk96 gtk96 deleted the DataStream_API_package_guidance branch November 26, 2023 05:39
e-mhui pushed a commit to e-mhui/flink-cdc-connectors that referenced this pull request Dec 2, 2023
This closes apache#2583.

Co-authored-by: Leonard Xu <leonard@apache.org>
ChaomingZhangCN pushed a commit to ChaomingZhangCN/flink-cdc that referenced this pull request Jan 13, 2025
This closes apache#2583.

Co-authored-by: Leonard Xu <leonard@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[docs] Add entire DataStream API package guidance

3 participants