morrySnow opened a new pull request, #69108: URL: https://github.com/apache/doris/pull/69108
### What problem does this PR solve? Issue Number: N/A Related PR: https://github.com/apache/doris/pull/67818 Problem Summary: Filtering unmatched CASE rows changes results for NULL-preserving aggregates. Gate aggregate CASE simplification on an explicit null-row contract. Backport the marker prerequisite for Count, Avg, Sum, Max and Min; omit aggregate classes absent from this branch and preserve the existing InferAggNotNull whitelist. Import upstream regression outputs without editing them, omitting only an explain assertion requiring the absent broader inference optimization. ### Release note Fix incorrect CASE simplification for NULL-preserving aggregate functions. ### Backport validation - Test: Unit Test: InferAggNotNullTest, SimplifyAggGroupByTest; FE Checkstyle (0 violations). - Combined eight-backport validation: 283 tests across 13 classes, 0 failures/errors after correcting identity-cast assertions. This standalone backport has not had a separate unit-test run. - Regression tests were not run: no isolated branch-4.2 cluster was deployed. - Full build.sh --fe packaging was blocked by incomplete native third-party dependencies; the prescribed FE unit-test entry point compiled FE sources. - Behavior changed: Yes (CASE simplification requires the null-row contract) - Does this need documentation: No (cherry picked from commit 19a6089e4f95684ca05c3826eeb92d1c5bb1ec25) ### Scope This is an independent backport of #67818 onto branch-4.2. The branch contains one commit and does not include the other seven picks. The 283-test result above was obtained from the combined backport tree; this standalone branch has not had a separate unit-test run. ### Check List (For Author) - Test: - [x] Unit Test: 283 targeted tests passed on the combined backport tree. - [x] Manual test: standalone patch matches the validated pick; FE Checkstyle passed on the combined tree. - [ ] Regression test: no isolated branch-4.2 cluster was deployed. - Behavior changed: see the backport validation details above. - 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]
