review feedback

Author: smalyshev

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

@@ -33,12 +33,7 @@
  * A Fork is a n-ary {@code Plan} where each child is a sub plan, e.g.
  * {@code FORK [WHERE content:"fox" ] [WHERE content:"dog"] }
  */
-public class Fork extends LogicalPlan
-    implements
-        PostAnalysisPlanVerificationAware,
-        TelemetryAware,
-        PipelineBreaker,
-        ExecutesOn.Coordinator {
+public class Fork extends LogicalPlan implements PostAnalysisPlanVerificationAware, TelemetryAware, ExecutesOn.Coordinator {
 
     public static final String FORK_FIELD = "_fork";
     public static final int MAX_BRANCHES = 8;

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

@@ -331,6 +331,7 @@ private void checkRemoteJoin(Failures failures) {
             }
             if (u instanceof Limit) {
                 // Limit is ok because it can be moved in by the optimizer
+                // We check LIMITs in LookupJoin pre-optimization so they are still not allowed there
                 return;
             }
             if (u instanceof PipelineBreaker || (u instanceof ExecutesOn ex && ex.executesOn() == ExecuteLocation.COORDINATOR)) {