fix: use !asc

Co-authored-by: Andrew Lamb <andrew@nerdnetworks.org>
apache · Sep 20, 2024 · 6d432a3 · 6d432a3
1 parent 95e0341
commit 6d432a3
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/datafusion/sql/src/statement.rs b/datafusion/sql/src/statement.rs
@@ -1147,7 +1147,7 @@ impl<'a, S: ContextProvider> SqlToRel<'a, S> {
                         schema,
                         planner_context,
                     )?;
-                    let nulls_first = ordered_expr.nulls_first.unwrap_or(true);
+                    let nulls_first = ordered_expr.nulls_first.unwrap_or(!asc);
                     let asc = ordered_expr.asc.unwrap_or(true);
                     let sort_expr = SortExpr::new(order_expr, asc, nulls_first);
                     results.push(sort_expr);