datafusion icon indicating copy to clipboard operation
datafusion copied to clipboard

Add More `Arc` to AggregateFunctionExpr

Open alamb opened this issue 1 year ago • 0 comments

Draft as I run benchmarks

Which issue does this PR close?

Follow on to https://github.com/apache/datafusion/pull/12950

Rationale for this change

While reviewing https://github.com/apache/datafusion/pull/12950 from @askalt I noticed a few places where deep copies of Schema was happening when we could have simply been copying Arcs

What changes are included in this PR?

Add some more Arc to improce performance

Are these changes tested?

Are there any user-facing changes?

alamb avatar Oct 18 '24 20:10 alamb