Add deprecation for synchronous execute method in the RuleExecutor

afoucret · afoucret · commit c55951139320 · 2025-07-07T18:01:03.000+02:00
diff --git a/x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/rule/RuleExecutor.java b/x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/rule/RuleExecutor.java
@@ -138,6 +138,8 @@ public Map<Batch<TreeType>, List<Transformation>> transformations() {
         }
     }
 
+
+    @Deprecated(forRemoval = true)
     protected final TreeType execute(TreeType plan) {
         // TODO: remove when all implementinc classes are using the async version.
         PlainActionFuture<ExecutionInfo> executionInfoFuture = new PlainActionFuture<>();

Original file line number	Diff line number	Diff line change
`@@ -138,6 +138,8 @@ public Map<Batch<TreeType>, List<Transformation>> transformations() {`
`138`	`138`	`}`
`139`	`139`	`}`
`140`	`140`
	`141`	`+`
	`142`	`+ @Deprecated(forRemoval = true)`
`141`	`143`	`protected final TreeType execute(TreeType plan) {`
`142`	`144`	`// TODO: remove when all implementinc classes are using the async version.`
`143`	`145`	`PlainActionFuture<ExecutionInfo> executionInfoFuture = new PlainActionFuture<>();`