Expand comment

Author: alex-spies

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

@@ -768,7 +768,8 @@ private PhysicalOperation planLookupJoin(LookupJoinExec join, LocalExecutionPlan
 
     private record MatchConfig(FieldAttribute.FieldName fieldName, int channel, DataType type) {
         private MatchConfig(FieldAttribute match, Layout.ChannelAndType input) {
-            // Note, this handles TEXT fields with KEYWORD subfields
+            // TODO: Using exactAttribute was supposed to handle TEXT fields with KEYWORD subfields - but we don't allow these in lookup
+            // indices, so the call to exactAttribute looks redundant now.
             this(match.exactAttribute().fieldName(), input.channel(), input.type());
         }
     }