Skip to content

Commit 1054caf

Browse files
committed
ESQL: Rename evaluators for FIRST and LAST
I'm going to be reusing the evaluators from FIRST_OVER_TIME and LAST_OVER_TIME for FIRST and LAST. This renames the evaluators themselves to indicate that they are the actual evaluators for FIRST and LAST. Relates to #108385
1 parent 311a722 commit 1054caf

File tree

31 files changed

+356
-370
lines changed

31 files changed

+356
-370
lines changed

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

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -900,28 +900,28 @@ tasks.named('stringTemplates').configure {
900900
}
901901

902902
// TODO: add {value}_over_time for other types: boolean, bytes_refs
903-
File valueOverTimeAggregatorInputFile = file("src/main/java/org/elasticsearch/compute/aggregation/X-ValueOverTimeAggregator.java.st")
903+
File valueByLongAggregatorInputFile = file("src/main/java/org/elasticsearch/compute/aggregation/X-ValueByLongAggregator.java.st")
904904
["First", "Last"].forEach { Occurrence ->
905905
{
906906
template {
907907
it.properties = addOccurrence(intProperties, Occurrence)
908-
it.inputFile = valueOverTimeAggregatorInputFile
909-
it.outputFile = "org/elasticsearch/compute/aggregation/${Occurrence}OverTimeIntAggregator.java"
908+
it.inputFile = valueByLongAggregatorInputFile
909+
it.outputFile = "org/elasticsearch/compute/aggregation/${Occurrence}IntByLongAggregator.java"
910910
}
911911
template {
912912
it.properties = addOccurrence(longProperties, Occurrence)
913-
it.inputFile = valueOverTimeAggregatorInputFile
914-
it.outputFile = "org/elasticsearch/compute/aggregation/${Occurrence}OverTimeLongAggregator.java"
913+
it.inputFile = valueByLongAggregatorInputFile
914+
it.outputFile = "org/elasticsearch/compute/aggregation/${Occurrence}LongByLongAggregator.java"
915915
}
916916
template {
917917
it.properties = addOccurrence(floatProperties, Occurrence)
918-
it.inputFile = valueOverTimeAggregatorInputFile
919-
it.outputFile = "org/elasticsearch/compute/aggregation/${Occurrence}OverTimeFloatAggregator.java"
918+
it.inputFile = valueByLongAggregatorInputFile
919+
it.outputFile = "org/elasticsearch/compute/aggregation/${Occurrence}FloatByLongAggregator.java"
920920
}
921921
template {
922922
it.properties = addOccurrence(doubleProperties, Occurrence)
923-
it.inputFile = valueOverTimeAggregatorInputFile
924-
it.outputFile = "org/elasticsearch/compute/aggregation/${Occurrence}OverTimeDoubleAggregator.java"
923+
it.inputFile = valueByLongAggregatorInputFile
924+
it.outputFile = "org/elasticsearch/compute/aggregation/${Occurrence}DoubleByLongAggregator.java"
925925
}
926926
}
927927
}
Lines changed: 2 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Lines changed: 2 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Lines changed: 2 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Lines changed: 2 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Lines changed: 2 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Lines changed: 2 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Lines changed: 2 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Lines changed: 2 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Lines changed: 7 additions & 7 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)