[Feature](agg) add sem agg function#57545
Conversation
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
|
run buildall |
TPC-DS: Total hot run time: 189981 ms |
ClickBench: Total hot run time: 27.6 s |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
FE Regression Coverage ReportIncrement line coverage |
fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/functions/agg/Sem.java
Outdated
Show resolved
Hide resolved
...-test/suites/query_p0/sql_functions/aggregate_functions/test_aggregate_all_functions2.groovy
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Pull Request Overview
This PR implements a new aggregate function sem (Standard Error of Mean) for calculating the standard error of a sample. The implementation follows the existing patterns for statistical aggregate functions in Apache Doris.
Key Changes:
- Added
semaggregate function implementation in backend (C++) and frontend (Java) - Registered the new function in the builtin aggregate functions registry
- Added comprehensive regression tests covering various data types, window functions, and distinct operations
Reviewed Changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| be/src/vec/aggregate_functions/aggregate_function_sem.h | Core C++ implementation of SEM aggregate function with data structure and computation logic |
| be/src/vec/aggregate_functions/aggregate_function_sem.cpp | Registration of SEM function with factory for numeric types |
| be/src/vec/aggregate_functions/aggregate_function_simple_factory.cpp | Added registration call for SEM function |
| fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/functions/agg/Sem.java | Java implementation of SEM function for query planning |
| fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/visitor/AggregateFunctionVisitor.java | Added visitor method for SEM function |
| fe/fe-core/src/main/java/org/apache/doris/catalog/BuiltinAggregateFunctions.java | Registered SEM in catalog of built-in functions |
| regression-test/suites/query_p0/sql_functions/aggregate_functions/test_aggregate_all_functions2.groovy | Added comprehensive test cases for SEM function |
| regression-test/data/query_p0/sql_functions/aggregate_functions/test_aggregate_all_functions2.out | Expected test output for SEM function tests |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/functions/agg/Sem.java
Outdated
Show resolved
Hide resolved
...-test/suites/query_p0/sql_functions/aggregate_functions/test_aggregate_all_functions2.groovy
Outdated
Show resolved
Hide resolved
...-test/suites/query_p0/sql_functions/aggregate_functions/test_aggregate_all_functions2.groovy
Show resolved
Hide resolved
|
run buildall |
ClickBench: Total hot run time: 27.84 s |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
|
run buildall |
TPC-DS: Total hot run time: 190781 ms |
ClickBench: Total hot run time: 27.8 s |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
FE UT Coverage ReportIncrement line coverage |
|
PR approved by at least one committer and no changes requested. |
|
PR approved by anyone and no changes requested. |
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/functions/agg/Sem.java
Outdated
Show resolved
Hide resolved
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
FE Regression Coverage ReportIncrement line coverage |
7405010
|
run buildall |
TPC-DS: Total hot run time: 190297 ms |
ClickBench: Total hot run time: 27.9 s |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
FE UT Coverage ReportIncrement line coverage |
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
FE Regression Coverage ReportIncrement line coverage |
|
run p0 |
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
FE Regression Coverage ReportIncrement line coverage |
|
PR approved by at least one committer and no changes requested. |
add sem agg function
add sem agg function
What problem does this PR solve?
add sem agg function
Issue Number: #48203
Problem Summary:
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)