Skip to content

[Bug] [connector-file] When the data source field is less than the target (Hive) field，it will throw null pointer exception#8150 #8200

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.

Sign up for GitHub

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

Jump to bottom
Merged
hailin0 merged 1 commit into from
Jan 4, 2025
Merged

[Bug] [connector-file] When the data source field is less than the target (Hive) field，it will throw null pointer exception#8150 #8200

hailin0 merged 1 commit into from
Jan 4, 2025

Conversation

@linjianchang
Copy link
Contributor

@linjianchang linjianchang commented Dec 3, 2024

Purpose of this pull request

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

@linjianchang linjianchang marked this pull request as draft December 4, 2024 03:46
@linjianchang linjianchang changed the title #8150 [Bug] [connector-file] When the data source field is less than the target (Hive) field，it will throw null pointer exception#8150 Dec 4, 2024
@linjianchang linjianchang marked this pull request as ready for review December 4, 2024 07:27
@github-actions github-actions bot added the CI&CD label Dec 4, 2024
@github-actions github-actions bot added CI&CD and removed CI&CD labels Dec 4, 2024
@linjianchang linjianchang reopened this Dec 4, 2024
@github-actions github-actions bot removed the CI&CD label Dec 4, 2024
@Hisoka-X
Copy link
Member

Hisoka-X commented Dec 4, 2024

Thanks @linjianchang ! Could you add an unit test?

[Bug] [connector-file] When the data source field is less than the ta…
9facb82 
…rget (Hive) field，it will throw null pointer exception#8150
@linjianchang linjianchang reopened this Jan 3, 2025
@linjianchang
Copy link
Contributor Author

linjianchang commented Jan 3, 2025

Thanks @linjianchang ! Could you add an unit test?

Unit tests have been added, please review again, thank you

@Hisoka-X
Copy link
Member

Hisoka-X commented Jan 3, 2025

CI: https://github.com/linjianchang/seatunnel/actions/runs/12593993935/job/35100811128

@hailin0 hailin0 merged commit 25b8a02 into apache:dev Jan 4, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hailin0 hailin0 hailin0 approved these changes

@Hisoka-X Hisoka-X Hisoka-X approved these changes

Assignees

No one assigned

Labels

approved connectors-v2 file reviewed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@linjianchang @Hisoka-X @hailin0