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) 
<[email protected]>
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()?)),

Reply via email to