Blizzara opened a new pull request, #15049: URL: https://github.com/apache/datafusion/pull/15049
## Which issue does this PR close? See https://github.com/apache/datafusion/issues/14123#issuecomment-2703526303 ## Rationale for this change People should use `invoke_with_args` instead. Calling `invoke_batch` for any DF-builtin function will fail at runtime with something like `Function X does not implement invoke but called` We should also consider just removing the old functions (invoke_batch, invoke, invoke_no_args), to clear up the trait. ## What changes are included in this PR? Mark `invoke_batch` as deprecated. It was already called so in the docstring, but the compiler doesn't read those. ## Are these changes tested? Existing tests ## Are there any user-facing changes? New deprecation -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
