Mryange opened a new pull request, #69715:
URL: https://github.com/apache/doris/pull/69715
### What problem does this PR solve?
`positive()` allocates and copies a result column even though it preserves
its input values and type. Reuse the immutable input column for non-Decimal
arguments to avoid this work.
### Release note
Reduce CPU and allocation overhead of non-Decimal `positive()` expressions.
### Check List (For Author)
- Test
- [x] Manual test: Built and ran the same patch on ARM. Checked
`count(positive(u64)), count(positive(f64))` over 20 million rows, plus
negative, zero, positive, NULL, floating-point and nullable aggregate inputs.
Results were correct. This checkout was not rebuilt.
- [x] Formatting, `git diff --check`, and `bash
build-support/check-build-hygiene.sh` passed.
- Behavior changed:
- [x] No.
- Does this need documentation?
- [x] No.
### Check List (For Reviewer who merge this PR)
- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
--
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]
