commit b65ac813d0c4de67ed81de7e24f78d693d799a07
author Marc Prud'hommeaux <marc@glimpse.io> Fri Feb 12 17:29:42 2021 -0500
committer Andy Grove <andygrove@nvidia.com> Fri Mar 12 13:15:51 2021 -0700
tree fe6ed8c4f299c33d91dd5e26004b9a6c69f3da6f
parent bb5f3057db1d5b9514954f75b6bf64b43bcfd1a3

ARROW-11586: [Rust][Datafusion] Remove force unwrap

Fix for https://issues.apache.org/jira/browse/ARROW-11586

Closes #9479 from marcprux/patch-3

Authored-by: Marc Prud'hommeaux <marc@glimpse.io>
Signed-off-by: Andrew Lamb <andrew@nerdnetworks.org>

rust/datafusion/src/sql/planner.rs

diff --git a/rust/datafusion/src/sql/planner.rs b/rust/datafusion/src/sql/planner.rs
index d2cbd7c..8d6ed7f 100644
--- a/rust/datafusion/src/sql/planner.rs
+++ b/rust/datafusion/src/sql/planner.rs
@@ -558,7 +558,7 @@
             .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