Mryange opened a new pull request, #70110:
URL: https://github.com/apache/doris/pull/70110
### What problem does this PR solve?
Issue Number: N/A
Problem Summary:
IFNULL rebuilds its result row by row when its first argument is a nullable
column, even if the current batch contains no NULLs. This causes unnecessary
allocations and string copies.
Inspect the null-map state and reuse the first argument for batches without
NULLs, preserving the result type's nullable wrapper. Keep the existing
all-NULL and mixed-NULL paths.
### Release note
Improve IFNULL performance for nullable input batches without NULLs.
### Check List (For Author)
- Test
- [ ] Regression test
- [ ] Unit Test
- [x] Manual test
The same modified source was built and tested on an ARM Release
environment. All 25 SQL checks matched before and after the change, covering
all-NULL, no-NULL and mixed batches, nullable/nonnullable results, numeric and
string inputs, empty input, and shared input columns. The target checkout was
not rebuilt; full BE UT and regression suites were not run. clang-format 16,
build hygiene, and git diff --check passed locally.
- Behavior changed:
- [x] No.
- [ ] Yes.
- Does this need documentation?
- [x] No.
- [ ] Yes.
### Check List (For Reviewer who merge this PR)
- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick 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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
