matthewmturner opened a new pull request #1243:
URL: https://github.com/apache/arrow-rs/pull/1243
# Which issue does this PR close?
No existing issue.
# Rationale for this change
In the context of https://github.com/apache/arrow-datafusion/pull/1685 I am
trying to use datafusions `ScalarValue` with the dyn scalar kernels. We now
use the `ToPrimitive` trait for the dyn scalar kernels and I am looking to
implement that on `ScalarValue` as well. While testing that locally the last
issue i came across was `ScalarValue` not implementing `Copy` which the dyn
scalar kernels require. I tried removing that constraint locally and had no
issues compiling / testing / linting so im thinking the constraint isnt needed.
@alamb FYI.
# What changes are included in this PR?
Removing `Copy` trait requirement on the dyn scalar kernels.
# Are there any user-facing changes?
<!---
If there are user-facing changes then we may require documentation to be
updated before approving the PR.
-->
<!---
If there are any breaking changes to public APIs, please add the `breaking
change` label.
-->
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
