Skip to content

Conversation

@alamb
Copy link
Contributor

@alamb alamb commented Jan 23, 2024

Which issue does this PR close?

Closes #8973

Rationale for this change

CI checks are failing on main: https://github.com/apache/arrow-datafusion/actions/runs/7628515989/job/20779877406

error: use of deprecated function `functions::make_scalar_function`: Implement your function directly in terms of ColumnarValue or use `ScalarUDF` instead
   --> datafusion/physical-expr/src/functions.rs:562:17
    |
562 |                 make_scalar_function(string_expressions::instr::<i32>)(args)
    |                 ^^^^^^^^^^^^^^^^^^^^
    |
    = note: `-D deprecated` implied by `-D warnings`
    = help: to override `-D warnings` add `#[allow(deprecated)]`

I believe this is a logical conflict between #8862 and #8878

What changes are included in this PR?

Fix logical conflicts

Are these changes tested?

yes

Are there any user-facing changes?

@github-actions github-actions bot added the physical-expr Changes to the physical-expr crates label Jan 23, 2024
@alamb alamb changed the title Fix clippy failures Fix clippy failures: error: use of deprecated function `functions::make_scalar_function Jan 23, 2024
@alamb alamb merged commit ee7ab0b into apache:main Jan 23, 2024
@alamb
Copy link
Contributor Author

alamb commented Jan 23, 2024

Thanks @viirya

@alamb alamb deleted the alamb/fix_clippy branch January 23, 2024 22:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

physical-expr Changes to the physical-expr crates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clippy failures on main: error: use of deprecated function functions::make_scalar_function`

2 participants