[arrow-datafusion] branch master updated: Make parse_physical_expr public (#5118)

[al...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

alamb 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 a218b7033 Make parse_physical_expr public (#5118)
a218b7033 is described below

commit a218b70334e68a7ea13898d9a5b3a2060a902096
Author: comphead <co...@users.noreply.github.com>
AuthorDate: Tue Jan 31 09:17:26 2023 -0800

    Make parse_physical_expr public (#5118)
---
 datafusion/proto/src/physical_plan/from_proto.rs | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/datafusion/proto/src/physical_plan/from_proto.rs b/datafusion/proto/src/physical_plan/from_proto.rs
index d1141c850..5669410a8 100644
--- a/datafusion/proto/src/physical_plan/from_proto.rs
+++ b/datafusion/proto/src/physical_plan/from_proto.rs
@@ -59,7 +59,15 @@ impl From<&protobuf::PhysicalColumn> for Column {
     }
 }
 
-pub(crate) fn parse_physical_expr(
+/// Parses a physical expression from a protobuf.
+///
+/// # Arguments
+///
+/// * `proto` - Input proto with physical expression node
+/// * `registry` - A registry knows how to build logical expressions out of user-defined function' names
+/// * `input_schema` - The Arrow schema for the input, used for determining expression data types
+///                    when performing type coercion.
+pub fn parse_physical_expr(
     proto: &protobuf::PhysicalExprNode,
     registry: &dyn FunctionRegistry,
     input_schema: &Schema,