make InList Display implementation deterministic

andygrove · Dec 14, 2024 · 449dcf0 · 449dcf0
1 parent bfabd48
commit 449dcf0
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/datafusion/expr/src/expr.rs b/datafusion/expr/src/expr.rs
@@ -1843,11 +1843,12 @@ fn rewrite_placeholder(expr: &mut Expr, other: &Expr, schema: &DFSchema) -> Resu
 #[macro_export]
 macro_rules! expr_vec_fmt {
     ( $ARRAY:expr ) => {{
-        $ARRAY
+        let mut expr_str = $ARRAY
             .iter()
             .map(|e| format!("{e}"))
-            .collect::<Vec<String>>()
-            .join(", ")
+            .collect::<Vec<String>>();
+        expr_str.sort();
+        expr_str.join(", ")
     }};
 }