Skip to content

Commit ad4831c

Browse files
dnhatndnhatn
authored
Use lossy summation for time-series aggregations (#132625)
Kahan summation can be expensive, and for time-series aggregation, a lossy summation can be a good trade-off for performance. This change introduces a lossy summation mode and makes it the default for time-series aggregations. These two summation modes for sum and avg are used internally and are not exposed to users.
1 parent df5cd30 commit ad4831c

File tree

17 files changed

+1197
-33
lines changed

17 files changed

+1197
-33
lines changed

‎x-pack/plugin/esql/compute/src/main/generated/org/elasticsearch/compute/aggregation/LossySumDoubleAggregatorFunction.java

Lines changed: 193 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎x-pack/plugin/esql/compute/src/main/generated/org/elasticsearch/compute/aggregation/LossySumDoubleAggregatorFunctionSupplier.java

Lines changed: 47 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)