You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by ho...@apache.org on 2022/02/15 06:15:21 UTC
[arrow-datafusion] branch master updated: Octetlength in protobuf should use the octet_length function (#1834)
This is an automated email from the ASF dual-hosted git repository.
houqp pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow-datafusion.git
The following commit(s) were added to refs/heads/master by this push:
new d79ca16 Octetlength in protobuf should use the octet_length function (#1834)
d79ca16 is described below
commit d79ca1632da4bbddaa650c9c44fb6ffcfa4572e7
Author: Carol (Nichols || Goulding) <19...@users.noreply.github.com>
AuthorDate: Tue Feb 15 01:15:14 2022 -0500
Octetlength in protobuf should use the octet_length function (#1834)
---
ballista/rust/core/src/serde/logical_plan/from_proto.rs | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/ballista/rust/core/src/serde/logical_plan/from_proto.rs b/ballista/rust/core/src/serde/logical_plan/from_proto.rs
index fb900e1..d4a28cf 100644
--- a/ballista/rust/core/src/serde/logical_plan/from_proto.rs
+++ b/ballista/rust/core/src/serde/logical_plan/from_proto.rs
@@ -880,7 +880,7 @@ impl TryInto<Expr> for &protobuf::LogicalExprNode {
Ok(signum((&args[0]).try_into()?))
}
protobuf::ScalarFunction::Octetlength => {
- Ok(length((&args[0]).try_into()?))
+ Ok(octet_length((&args[0]).try_into()?))
}
// // protobuf::ScalarFunction::Concat => Ok(concat((&args[0]).try_into()?)),
protobuf::ScalarFunction::Lower => Ok(lower((&args[0]).try_into()?)),