Skip to content

Migrate invoke_no_args to invoke_batch#13246

Merged
alamb merged 1 commit into
apache:mainfrom
findepi:findepi/migrate-invoke-no-args-to-invoke-batch-95828d
Nov 5, 2024
Merged

Migrate invoke_no_args to invoke_batch#13246
alamb merged 1 commit into
apache:mainfrom
findepi:findepi/migrate-invoke-no-args-to-invoke-batch-95828d

Conversation

@findepi

@findepi findepi commented Nov 4, 2024

Copy link
Copy Markdown
Member

@github-actions github-actions Bot added logical-expr Logical plan and expressions core Core DataFusion crate functions Changes to functions implementation labels Nov 4, 2024
@findepi

findepi commented Nov 4, 2024

Copy link
Copy Markdown
Member Author

Currently based on #13174

@findepi findepi force-pushed the findepi/migrate-invoke-no-args-to-invoke-batch-95828d branch from 7f49eb8 to 80d3cc7 Compare November 4, 2024 12:55
Comment thread datafusion/expr/src/udf.rs Outdated

@comphead comphead 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.

lgtm thanks @findepi

@findepi

findepi commented Nov 4, 2024

Copy link
Copy Markdown
Member Author

let me rebase after #13174 merged

Migrate scalar UDFs from deprecated invoke_no_args to invoke_batch.
@findepi findepi force-pushed the findepi/migrate-invoke-no-args-to-invoke-batch-95828d branch from ba718e4 to 2b08be3 Compare November 4, 2024 19:47
@alamb

alamb commented Nov 5, 2024

Copy link
Copy Markdown
Contributor

🚀

@alamb

alamb commented Nov 5, 2024

Copy link
Copy Markdown
Contributor

THanks again @findepi and @comphead

@alamb alamb merged commit e43466a into apache:main Nov 5, 2024
@findepi findepi deleted the findepi/migrate-invoke-no-args-to-invoke-batch-95828d branch November 5, 2024 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Core DataFusion crate functions Changes to functions implementation logical-expr Logical plan and expressions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants