-
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
[Feature][Oracle] Support XMLTYPE data integration #5716 #5723
Conversation
@Hisoka-X |
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.
Could you add a e2e test case for this feature?
Sure. I'll add one later |
@Hisoka-X Sorry for the busy work later. I have completed the PR test by modifying the previous Orcale E2E and improved the previous submission. Please kindly help me review it |
Hi @SimonChou12138 , could you retrigger the failed CI? https://github.com/SimonChou12138/seatunnel/actions/runs/6715296342/job/18249846406 |
Hi @Hisoka-X |
It's unstable. Don't worry it. Just retigger it after failed. |
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 @SimonChou12138 , just one question. Why one called XMLTYPE
and another called SYS.XMLTYPE
?
Hi @Hisoka-X That's a good question, and I actually had the same question at first. This type is actually read in different places, one is Catalog and the other is JDBC Row. After Oracle defines the type 'SYS.XMLTYPE', you will find that the DDL field will be 'XMLTYPE', but it will be 'SYS.XMLTYPE' when JDBC Row reads it. |
|
Purpose of this pull request
Implement Feature #5716
Does this PR introduce any user-facing change?
without
How was this patch tested?
Create an Oracle A table
New data
Create an Oracle B table
Build a collection profile
result
SEATUNEL_ORACLE_TEST_B table data is synchronized successfully
Check list
New License Guide
release-note
.