This is an automated email from the ASF dual-hosted git repository.
agrove pushed a commit to branch maint-3.0.x
in repository https://gitbox.apache.org/repos/asf/arrow.git
The following commit(s) were added to refs/heads/maint-3.0.x by this push:
new b65ac81 ARROW-11586: [Rust][Datafusion] Remove force unwrap
b65ac81 is described below
commit b65ac813d0c4de67ed81de7e24f78d693d799a07
Author: Marc Prud'hommeaux <[email protected]>
AuthorDate: Fri Feb 12 17:29:42 2021 -0500
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 <[email protected]>
Signed-off-by: Andrew Lamb <[email protected]>
---
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 d2cbd7c..8d6ed7f 100644
--- a/rust/datafusion/src/sql/planner.rs
+++ b/rust/datafusion/src/sql/planner.rs
@@ -558,7 +558,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
