I bet you can make this even faster by making it mutate lhs rather than make a new one ```rust pub(crate) fn scalar_add(lhs: &mut ScalarValue, rhs: &ScalarValue) -> Result<()> { ``` _Originally posted by @alamb in https://github.com/apache/datafusion/pull/20768#discussion_r2924417922_ The other idea is to inline the implementation of precision_add to ScalarValue::add for numeric types, in order to avoid allocations when adding
Activity
perf: add in-place fast path for ScalarValue::add (#20959)
perf: add in-place fast path for ScalarValue::add (apache#20959)
perf: add in-place fast path for ScalarValue::add (apache#20959)