upd

Author: idegtiarenko

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

@@ -156,10 +156,26 @@ private static Object doTransformExpression(Object arg, Function<Expression, ? e
                     transformed.set(i, next);
                 }
             }
-
+            return hasChanged ? transformed : arg;
+        } else if (arg instanceof Collection<?> c) {
+            List<Object> transformed = null;
+            boolean hasChanged = false;
+            int i = 0;
+            for (Object e : c) {
+                Object next = doTransformExpression(e, traversal);
+                if (e.equals(next) == false) {
+                    if (hasChanged == false) {
+                        hasChanged = true;
+                        // TODO if collection is a set then this silently changes its semantics by allowing duplicates
+                        // We should fix it or confirm this branch is never needed
+                        transformed = new ArrayList<>(c);
+                    }
+                    transformed.set(i, next);
+                }
+                i++;
+            }
             return hasChanged ? transformed : arg;
         }
-        assert arg instanceof Collection<?> == false : "Non-List Collection implementations are not supported";
 
         return arg;
     }