mrhhsg opened a new pull request, #68509:
URL: https://github.com/apache/doris/pull/68509
### What problem does this PR solve?
Issue Number: None
Related PR: #68318 #68475
Problem Summary: The backport of #68318 to branch-4.1 (#68475) also brought
the regression case `query_p0/aggregate/percentile_bucketed_agg_merge`. The
case targets the bucketed hash aggregation operator and sets
`enable_bucketed_hash_agg=true` plus the `bucketed_agg_*` session variables.
Bucketed hash aggregation (#61495) only exists on master; branch-4.1 has
neither the operator nor these session variables. Every P0 and cloud P0 run
of a branch-4.1 PR therefore fails deterministically at
`set enable_bucketed_hash_agg=true`:
Unknown system variable 'enable_bucketed_hash_agg'
The source-side state merge that the case protects does not exist on
branch-4.1, so there is nothing for the case to cover there. The percentile
merge fix itself stays covered by the backported BE unit test
`be/test/util/percentile_util_test.cpp`. Remove the case and its expected
output from branch-4.1 only; master keeps it.
### Release note
None
### Check List (For Author)
- Test:
- No need to test: test-only removal. Verified that branch-4.1 has no
`enable_bucketed_hash_agg` / `bucketed_agg_*` session variable and no
bucketed aggregation operator, and that nothing else references the
removed case.
- Behavior changed: No
- 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]
