-
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] StarRocks-sink support schema evolution #8082
base: dev
Are you sure you want to change the base?
Conversation
…ing with Elasticsearch's automatic index creation apache#7430
…utomatic index creation conflict apache#7430
…utomatically creating indexes based on templates apache#7430
…utomatic index creation conflict apache#7430
…utomatic index creation conflict apache#7430
} | ||
|
||
@TestTemplate | ||
public void testStarRocksSinkWithSchemaEvolutionCase(TestContainer container) |
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.
reference
Line 119 in 69cd4ae
public void testMysqlCdcWithSchemaEvolutionCase(TestContainer container) |
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.
ok, thanks
@hailin0 hi,can you help me see the reason why CI did not pass,thanks |
@sunxiaojian thank you very much! I know how to view the logs now. I didn't find any errors in the action interface before |
} | ||
|
||
try { | ||
Connection conn = |
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.
Why not close connection ?
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.
Why not close connection ?
sorry, I'll optimize it
if (event instanceof AlterTableColumnsEvent) { | ||
AlterTableColumnsEvent alterTableColumnsEvent = (AlterTableColumnsEvent) event; |
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.
reference
#8158
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.
ok,thanks!
.../main/java/org/apache/seatunnel/connectors/seatunnel/starrocks/sink/StarRocksSinkWriter.java
Outdated
Show resolved
Hide resolved
Please fix the ci error |
ok! |
Feature #8034
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
.