Skip to content

Commit 5179b09

Browse files
bpintearjernst
authored andcommitted
Shorten method name in PostOptimizationVerificationAware (elastic#120307)
This simply renames a method in the interface: s/postLogicalOptimizationVerification/postOptimizationVerification It also drops a javadoc reference.
1 parent 6cdffc0 commit 5179b09

File tree

7 files changed

+9
-10
lines changed

7 files changed

+9
-10
lines changed

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/capabilities/PostOptimizationVerificationAware.java

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@
88
package org.elasticsearch.xpack.esql.capabilities;
99

1010
import org.elasticsearch.xpack.esql.common.Failures;
11-
import org.elasticsearch.xpack.esql.expression.function.grouping.Bucket;
1211

1312
/**
1413
* Interface implemented by expressions that require validation post logical optimization,
@@ -21,13 +20,13 @@ public interface PostOptimizationVerificationAware {
2120
* {@link Failures} class.
2221
*
2322
* <p>
24-
* Example: the {@link Bucket} function, which produces buckets over a numerical or date field, based on a number of literal
23+
* Example: the {@code Bucket} function, which produces buckets over a numerical or date field, based on a number of literal
2524
* arguments needs to check if its arguments are all indeed literals. This is how this verification is performed:
2625
* <pre>
2726
* {@code
2827
*
2928
* @Override
30-
* public void postLogicalOptimizationVerification(Failures failures) {
29+
* public void postOptimizationVerification(Failures failures) {
3130
* String operation = sourceText();
3231
*
3332
* failures.add(isFoldable(buckets, operation, SECOND))
@@ -38,5 +37,5 @@ public interface PostOptimizationVerificationAware {
3837
* </pre>
3938
*
4039
*/
41-
void postLogicalOptimizationVerification(Failures failures);
40+
void postOptimizationVerification(Failures failures);
4241
}

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/fulltext/Match.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -203,7 +203,7 @@ protected TypeResolution checkParamCompatibility() {
203203
}
204204

205205
@Override
206-
public void postLogicalOptimizationVerification(Failures failures) {
206+
public void postOptimizationVerification(Failures failures) {
207207
Expression fieldExpression = field();
208208
// Field may be converted to other data type (field_name :: data_type), so we need to check the original field
209209
if (fieldExpression instanceof AbstractConvertFunction convertFunction) {

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/fulltext/Term.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ protected TypeResolution resolveNonQueryParamTypes() {
100100
}
101101

102102
@Override
103-
public void postLogicalOptimizationVerification(Failures failures) {
103+
public void postOptimizationVerification(Failures failures) {
104104
if (field instanceof FieldAttribute == false) {
105105
failures.add(
106106
Failure.fail(

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/grouping/Bucket.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -409,7 +409,7 @@ private static TypeResolution isStringOrDate(Expression e, String operationName,
409409
}
410410

411411
@Override
412-
public void postLogicalOptimizationVerification(Failures failures) {
412+
public void postOptimizationVerification(Failures failures) {
413413
String operation = sourceText();
414414

415415
failures.add(isFoldable(buckets, operation, SECOND))

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/scalar/convert/FoldablesConvertFunction.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ public final Object fold(FoldContext ctx) {
7171
}
7272

7373
@Override
74-
public final void postLogicalOptimizationVerification(Failures failures) {
74+
public final void postOptimizationVerification(Failures failures) {
7575
failures.add(isFoldable(field(), sourceText(), null));
7676
}
7777
}

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/scalar/multivalue/MvSort.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -231,7 +231,7 @@ public DataType dataType() {
231231
}
232232

233233
@Override
234-
public void postLogicalOptimizationVerification(Failures failures) {
234+
public void postOptimizationVerification(Failures failures) {
235235
if (order == null) {
236236
return;
237237
}

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/optimizer/LogicalVerifier.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ public Failures verify(LogicalPlan plan) {
2929
if (failures.hasFailures() == false) {
3030
p.forEachExpression(ex -> {
3131
if (ex instanceof PostOptimizationVerificationAware va) {
32-
va.postLogicalOptimizationVerification(failures);
32+
va.postOptimizationVerification(failures);
3333
}
3434
});
3535
}

0 commit comments

Comments
 (0)