-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[Fix][Docs] Update jdbc doc and kafka doc #4380
Conversation
url = "jdbc:mysql://localhost:3306/test" | ||
url = "jdbc:mysql://localhost:3306" | ||
driver = "com.mysql.cj.jdbc.Driver" | ||
user = "root" | ||
password = "123456" | ||
|
||
table = sink_table | ||
database = "sink_database" | ||
table = "sink_table" | ||
primary_keys = ["key1", "key2", ...] |
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.
What's the advantage of this change?
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.
What's the advantage of this change?
The parameters of the dev branch connector have been changed, but not in the doc examples.These two changes were discovered when I was using the dev branch of SeaTunnel.
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.
LGTM, thansk for your contribution.
It's my pleasure. |
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.
LGTM
Co-authored-by: chaorongzhi <[email protected]>
Co-authored-by: chaorongzhi <[email protected]>
Purpose of this pull request
The doc of next vsersion has some problems, this pull request fixing some issues with the jdbc and kafka doc.
Check list
New License Guide
release-note
.