-
Notifications
You must be signed in to change notification settings - Fork 2.2k
[Fix][zeta][Connector-paimon] Fix NullPointerException when restore from checkpoint in paimon task #8730
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
Conversation
update close error msg
|
Please follow the link to open github action on your fork repositoy. https://github.com/apache/seatunnel/pull/8730/checks?check_run_id=37219576667 |
I have checked and passed |
Hisoka-X
left a comment
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 update PaimonSaveModeHandler too. I think we can skip setLoadTable in PaimonSaveModeHandler if paimonTable field value is not null.
|
Could you add a test case too? |
It has been modified. If it is RECREATE_SCHEMA mode, it needs to be set |
|
This modification will no longer report null pointer errors when running from checkpoint, but the Paimon jar package code will report error " File deletion conflicts detected! Give up committing. so test cases cannot be added. It is estimated that a new version of Paimon jar will solve this problem |
|
cc @hawk9821 as well. |
hawk9821
left a comment
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.
does version 2.3.9 reliably reproduce this issue ?
@hawk9821 2.3.9 and master branch having the same issue |
refer to MysqlCDCIT.testMultiTableWithRestore e2e case , could you add a test case ? |
|
Hisoka-X
left a comment
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.
@uniding Paimon version upgrade to 1.1.1 #8074 , please add test case |
|
This pull request has been automatically marked as stale because it has not had recent activity for 120 days. It will be closed in 7 days if no further activity occurs. |
|
This pull request has been closed because it has not had recent activity. You could reopen it if you try to continue your work, and anyone who are interested in it are encouraged to continue work on this pull request. |
|
why
Sent from Proton Mail for iOS
On 周日, 12月 7, 2025 08:41, github-actions[bot] ***@***.***> 来信:
… Closed [#8730](#8730).
—
Reply to this email directly, [view it on GitHub](#8730 (comment)), or [unsubscribe](https://github.com/notifications/unsubscribe-auth/A3XDNAW475OS33NCPP7Q3ND4ANZSLAVCNFSM6AAAAABXEFBJOKVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMRRGQYDMMZXG42TCMA).
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
Purpose of this pull request
close #8460 #8070
Does this PR introduce any user-facing change?
How was this patch tested?
Check list
New License Guide
release-note.