From ed800b758171c9348a9dd7bde126106c49ee949f Mon Sep 17 00:00:00 2001 From: Marc Prud'hommeaux Date: Fri, 12 Feb 2021 08:30:40 -0500 Subject: [PATCH] ARROW-11586: [Rust][Datafusion][WIP] Remove force unwrap Fix for https://issues.apache.org/jira/browse/ARROW-11586 --- rust/datafusion/src/sql/planner.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/rust/datafusion/src/sql/planner.rs b/rust/datafusion/src/sql/planner.rs index 5d505fd2ec6..58310f50856 100644 --- a/rust/datafusion/src/sql/planner.rs +++ b/rust/datafusion/src/sql/planner.rs @@ -649,7 +649,7 @@ impl<'a, S: ContextProvider> SqlToRel<'a, S> { .iter() .map(|e| { Ok(Expr::Sort { - expr: Box::new(self.sql_to_rex(&e.expr, &input_schema).unwrap()), + expr: Box::new(self.sql_to_rex(&e.expr, &input_schema)?), // by default asc asc: e.asc.unwrap_or(true), // by default nulls first to be consistent with spark