-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[Feature][Connector-V2] sftp file sink suport multiple table and save mode #7668
[Feature][Connector-V2] sftp file sink suport multiple table and save mode #7668
Conversation
@TestTemplate | ||
@DisabledOnContainer( | ||
value = {}, | ||
type = {EngineType.FLINK, EngineType.SPARK}, |
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.
type = {EngineType.FLINK, EngineType.SPARK}, | |
type = {EngineType.FLINK}, |
@@ -0,0 +1,103 @@ | |||
# |
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.
Please rename the test case file name. I can not get the difference of two config from config name.
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.
updated
@DisabledOnContainer( | ||
value = {}, | ||
type = {EngineType.FLINK}, | ||
disabledReason = "Flink dosen't support multi-table at now") |
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.
Flink supported now.
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.
updated
Purpose of this pull request
Does this PR introduce any user-facing change?
How was this patch tested?
Check list
New License Guide
release-note
.