BiteTheDDDDt opened a new pull request, #63279:
URL: https://github.com/apache/doris/pull/63279
### What problem does this PR solve?
Issue Number: None
Related PR: None
Problem Summary: Storage common expressions are prepared with scan tuple
column positions, but SegmentIterator evaluates them on storage reader blocks
whose schema can be expanded for aggregation/merge readers. This can make slot
refs read the wrong column and trigger type mismatches such as BIGINT vs INT in
eq predicates.
### Release note
None
### Check List (For Author)
- Test: Regression test / Manual test
- Added regression case for AGG key common expr with cast predicate.
- Ran build-support/clang-format.sh and build-support/check-format.sh.
- Ran syntax-only compile for
be/src/storage/segment/segment_iterator.cpp and
be/src/exec/scan/olap_scanner.cpp.
- Ran build-support/run-clang-tidy.sh --build-dir be/ut_build_ASAN; it
reports no warnings in the new segment_iterator.cpp helper, but still fails on
existing diagnostics and jni-util.h static_assert in this environment.
- Behavior changed: Yes. Fix storage common expr evaluation to use
SegmentIterator reader schema column positions.
- Does this need documentation: No
--
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]
