Skip to content

Commit 63b6d96

Browse files
authored
Merge old rate implementation to new rate (#134603)
This change merges the old rate implementation into the new one, allowing us to apply optimizations in follow-up PRs. It also fixes issues between the old and new implementations introduced in the previous PR.
1 parent b1e1a54 commit 63b6d96

14 files changed

+2841
-3779
lines changed

x-pack/plugin/esql/compute/build.gradle

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -953,4 +953,21 @@ tasks.named('stringTemplates').configure {
953953
}
954954
}
955955
}
956+
957+
File rateAggregatorInputFile = file("src/main/java/org/elasticsearch/compute/aggregation/X-RateGroupingAggregatorFunction.java.st")
958+
template {
959+
it.properties = intProperties
960+
it.inputFile = rateAggregatorInputFile
961+
it.outputFile = "org/elasticsearch/compute/aggregation/RateIntGroupingAggregatorFunction.java"
962+
}
963+
template {
964+
it.properties = doubleProperties
965+
it.inputFile = rateAggregatorInputFile
966+
it.outputFile = "org/elasticsearch/compute/aggregation/RateDoubleGroupingAggregatorFunction.java"
967+
}
968+
template {
969+
it.properties = longProperties
970+
it.inputFile = rateAggregatorInputFile
971+
it.outputFile = "org/elasticsearch/compute/aggregation/RateLongGroupingAggregatorFunction.java"
972+
}
956973
}

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

Lines changed: 706 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-src/org/elasticsearch/compute/aggregation/RateIntGroupingAggregatorFunction.java

Lines changed: 706 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-src/org/elasticsearch/compute/aggregation/RateLongGroupingAggregatorFunction.java

Lines changed: 706 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)