fix sqllogictest

Signed-off-by: Runji Wang <[email protected]>

src/binder/mod.rs

@@ -364,7 +364,11 @@ impl Binder {
     /// Add a CTE to the current context.
     fn add_cte(&mut self, table_name: &str, query: Id, columns: HashMap<String, Id>) -> Result<()> {
         let context = self.contexts.last_mut().unwrap();
-        if context.ctes.insert(table_name.into(), (query, columns)).is_some() {
+        if context
+            .ctes
+            .insert(table_name.into(), (query, columns))
+            .is_some()
+        {
             return Err(BindError::DuplicatedCteName(table_name.into()));
         }
         Ok(())

src/binder/select.rs

@@ -330,7 +330,7 @@ impl Binder {
         }
         let aggs = self.egraph.add(Node::List(aggs.into()));
         *projection = self.egraph.add(Node::List(projs.into()));
-        Ok(self.egraph.add(Node::HashAgg([aggs, distinct, plan])))
+        Ok(self.egraph.add(Node::HashAgg([distinct, aggs, plan])))
     }
 
     /// Extracts all over nodes from `projection`, `distinct` and `orderby`.

tests/sql/join_left_inner.slt

@@ -57,7 +57,7 @@ create table b(v3 int, v4 int);
 statement ok
 insert into a values (1, 1), (2, 2), (3, 3);
 
-query IIII
+query IIII rowsort
 select v1, v2, v3, v4 from a left join b on v1 = v3;
 ----
 1 1 NULL NULL