Skip to content

Commit d66e33f

Browse files
committed
fix merge
1 parent f853b57 commit d66e33f

File tree

1 file changed

+4
-12
lines changed
  • x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plan/logical

1 file changed

+4
-12
lines changed

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plan/logical/Enrich.java

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,8 @@
1717
import org.elasticsearch.index.IndexMode;
1818
import org.elasticsearch.transport.RemoteClusterAware;
1919
import org.elasticsearch.xpack.core.enrich.EnrichPolicy;
20+
import org.elasticsearch.xpack.esql.capabilities.PostAnalysisPlanVerificationAware;
2021
import org.elasticsearch.xpack.esql.capabilities.PostAnalysisVerificationAware;
21-
import org.elasticsearch.xpack.esql.capabilities.PostOptimizationVerificationAware;
2222
import org.elasticsearch.xpack.esql.capabilities.TelemetryAware;
2323
import org.elasticsearch.xpack.esql.common.Failures;
2424
import org.elasticsearch.xpack.esql.core.capabilities.Resolvables;
@@ -55,11 +55,10 @@
5555
public class Enrich extends UnaryPlan
5656
implements
5757
GeneratingPlan<Enrich>,
58-
PostOptimizationVerificationAware,
58+
PostAnalysisPlanVerificationAware,
5959
PostAnalysisVerificationAware,
6060
TelemetryAware,
61-
SortAgnostic,
62-
ExecutesOn {
61+
SortAgnostic {
6362
public static final NamedWriteableRegistry.Entry ENTRY = new NamedWriteableRegistry.Entry(
6463
LogicalPlan.class,
6564
"Enrich",
@@ -332,7 +331,7 @@ private static void checkForPlansForbiddenBeforeRemoteEnrich(Enrich enrich, Fail
332331
}
333332
});
334333

335-
fails.forEach(f -> failures.add(fail(this, "ENRICH with remote policy can't be executed after [" + f.text() + "]" + f.source())));
334+
badCommands.forEach(c -> failures.add(fail(enrich, "ENRICH with remote policy can't be executed after " + c)));
336335
}
337336

338337
/**
@@ -364,11 +363,4 @@ public void postAnalysisVerification(Failures failures) {
364363
}
365364

366365
}
367-
368-
@Override
369-
public void postOptimizationVerification(Failures failures) {
370-
if (this.mode == Mode.REMOTE) {
371-
checkForPlansForbiddenBeforeRemoteEnrich(failures);
372-
}
373-
}
374366
}

0 commit comments

Comments
 (0)