Skip to content

Minor: Refactor function argument handling in ScalarFunctionDefinition - #8387

Merged
alamb merged 1 commit into
apache:mainfrom
Weijun-H:refactor-scalar
Dec 1, 2023
Merged

Minor: Refactor function argument handling in ScalarFunctionDefinition#8387
alamb merged 1 commit into
apache:mainfrom
Weijun-H:refactor-scalar

Conversation

@Weijun-H

Copy link
Copy Markdown
Member

Which issue does this PR close?

Closes #.

Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

@github-actionsgithub-actionsBot added logical-expr Logical plan and expressions physical-expr Changes to the physical-expr crates labels Dec 1, 2023

@alambalamb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👨‍🍳 👌 -- very nice @Weijun-H -- thank you for the cleanup

@alamb
alamb merged commit a6e6d3f into apache:mainDec 1, 2023
appletreeisyellow pushed a commit to appletreeisyellow/datafusion that referenced this pull request Dec 14, 2023
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

logical-exprLogical plan and expressionsphysical-exprChanges to the physical-expr crates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@Weijun-H@alamb