Fix comparison between expressions and attributes

Author: costin

x-pack/plugin/sql/src/main/java/org/elasticsearch/xpack/sql/analysis/analyzer/Verifier.java

@@ -243,7 +243,9 @@ private static boolean checkGroupByOrder(LogicalPlan p, Set<Failure> localFailur
                         return;
                     }
 
-                    if (Expressions.anyMatch(a.groupings(), e::semanticEquals)) {
+                    // make sure to compare attributes directly
+                    if (Expressions.anyMatch(a.groupings(), 
+                            g -> e.semanticEquals(e instanceof Attribute ? Expressions.attribute(g) : g))) {
                         return;
                     }