Skip to content

Commit cdebea5

Browse files
committed
fix comment
1 parent 39caa37 commit cdebea5

File tree

1 file changed

+4
-1
lines changed
  • x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/planner/mapper

1 file changed

+4
-1
lines changed

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/planner/mapper/Mapper.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,10 @@ private PhysicalPlan mapUnary(UnaryPlan unary) {
120120
return unaryExec.child();
121121
}
122122
}
123-
// Currently, it's either UnaryExec or LeafExec. Leaf will either resolve to FragmentExec or we'll ignore it.
123+
// Here we have the following possibilities:
124+
// 1. LeafExec - should resolve to FragmentExec or we can ignore it
125+
// 2. Join - must be remote, and thus will go inside FragmentExec
126+
// 3. Fork/MergeExec - not currently allowed with remote enrich
124127
return f;
125128
});
126129

0 commit comments

Comments
 (0)