AdamGS opened a new pull request, #21219:
URL: https://github.com/apache/datafusion/pull/21219
## Which issue does this PR close?
Follow up of #20768.
## Rationale for this change
`Precision::min/max` allocates a lot of new `ScalarValues`, and it can be
done in place.
While running the `sql_planner` benchmark, it seems like for clickbench
`Statistics::try_merge_iter` is a significant part of the runtime, and this PR
improves that part by about 20-25% locally.
## What changes are included in this PR?
Introduces a couple of of new internal functions to calculate the min/max of
a `Precision` in-place.
## Are these changes tested?
Existing general tests, and a few new unit tests.
## Are there any user-facing changes?
None
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
