Skip to content

Commit c10b6ce

Browse files
authored
ESQL: Fix AggregatorFunctionSupplierImplementer warnings import (#121654) (#121668)
Extracted from https://github.com/elastic/elasticsearch/pull/116170/files#diff-d8dcb5e856b601ac28b3dc0055e09d5c815d2577705eade446bd55b2fca42ac7R145 Those warnings are not being used in main yet, so there's no error, and that code wasn't being executed. But it better be fixed in main already
1 parent 9ca2890 commit c10b6ce

File tree

1 file changed

+7
-4
lines changed

1 file changed

+7
-4
lines changed

x-pack/plugin/esql/compute/gen/src/main/java/org/elasticsearch/compute/gen/AggregatorFunctionSupplierImplementer.java

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@
3333
import static org.elasticsearch.compute.gen.Types.DRIVER_CONTEXT;
3434
import static org.elasticsearch.compute.gen.Types.LIST_INTEGER;
3535
import static org.elasticsearch.compute.gen.Types.STRING;
36+
import static org.elasticsearch.compute.gen.Types.WARNINGS;
3637

3738
/**
3839
* Implements "AggregationFunctionSupplier" from a class annotated with both
@@ -139,8 +140,9 @@ private MethodSpec aggregator() {
139140

140141
if (hasWarnings) {
141142
builder.addStatement(
142-
"var warnings = Warnings.createWarnings(driverContext.warningsMode(), "
143-
+ "warningsLineNumber, warningsColumnNumber, warningsSourceText)"
143+
"var warnings = $T.createWarnings(driverContext.warningsMode(), "
144+
+ "warningsLineNumber, warningsColumnNumber, warningsSourceText)",
145+
WARNINGS
144146
);
145147
}
146148

@@ -164,8 +166,9 @@ private MethodSpec groupingAggregator() {
164166

165167
if (hasWarnings) {
166168
builder.addStatement(
167-
"var warnings = Warnings.createWarnings(driverContext.warningsMode(), "
168-
+ "warningsLineNumber, warningsColumnNumber, warningsSourceText)"
169+
"var warnings = $T.createWarnings(driverContext.warningsMode(), "
170+
+ "warningsLineNumber, warningsColumnNumber, warningsSourceText)",
171+
WARNINGS
169172
);
170173
}
171174

0 commit comments

Comments
 (0)