Skip to content

Conversation

@hawk9821
Copy link
Contributor

Purpose of this pull request

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

@github-actions github-actions bot added the core SeaTunnel core module label Aug 24, 2024
@Hisoka-X Hisoka-X changed the title [hotfix][zeta] conf property is null, print log bug fix [Fix][Core] Fix NPE when config value is null Aug 24, 2024
Copy link
Member

@Hisoka-X Hisoka-X left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM if ci passes. Thanks @hawk9821

Copy link
Member

@wuchunfu wuchunfu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wuchunfu wuchunfu merged commit 2f31404 into apache:dev Aug 24, 2024
hawk9821 added a commit to hawk9821/seatunnel that referenced this pull request Aug 29, 2024
hawk9821 added a commit to hawk9821/seatunnel that referenced this pull request Aug 29, 2024
@hawk9821 hawk9821 deleted the log_bugfix branch September 2, 2024 06:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved core SeaTunnel core module reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants