morrySnow opened a new pull request, #64885:
URL: https://github.com/apache/doris/pull/64885
### What problem does this PR solve?
Problem Summary: Bool aggregate functions such as bool_and, bool_or, and
bool_xor extend NullableAggregateFunction, but their AggregateFunctionVisitor
default methods delegated directly to visitAggregateFunction. This bypassed
visitor implementations that specialize NullableAggregateFunction. Route these
bool aggregate visitor methods through visitNullableAggregateFunction and add a
unit test that verifies all current NullableAggregateFunction defaults dispatch
to the nullable visitor path.
### Release note
None
### Check List (For Author)
- Test: Unit Test
- ./run-fe-ut.sh --run
org.apache.doris.nereids.trees.expressions.visitor.AggregateFunctionVisitorTest
- cd fe && mvn checkstyle:check -pl fe-core
- Behavior changed: Yes. Nullable bool aggregate functions now use the
NullableAggregateFunction visitor default path.
- 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]
