Skip to content

Conversation

@ueshin
Copy link
Member

@ueshin ueshin commented Oct 30, 2025

What changes were proposed in this pull request?

Supports logging in UDTFs.

Why are the changes needed?

The basic logging infrastructure was introduced in #52689, and UDTFs should also support logging.

Here adding support for UDTFs.

Does this PR introduce any user-facing change?

Yes, the logging feature will be available in UDTFs.

How was this patch tested?

Added the related tests.

Was this patch authored or co-authored using generative AI tooling?

No.

@ueshin
Copy link
Member Author

ueshin commented Oct 30, 2025

The driver-side worker (analyze for UDTFs, Python DataSource, etc.) support will be in the following PR.

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

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

+1, LGTM. Thank you, @ueshin .

@HyukjinKwon
Copy link
Member

Merged to master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants