You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by ji...@apache.org on 2022/02/09 10:09:51 UTC
[arrow-datafusion] branch logical-plan created (now 0d89120)
This is an automated email from the ASF dual-hosted git repository.
jiayuliu pushed a change to branch logical-plan
in repository https://gitbox.apache.org/repos/asf/arrow-datafusion.git.
at 0d89120 remove reference to physical plan in logical plan
This branch includes the following new commits:
new 0d89120 remove reference to physical plan in logical plan
The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
[arrow-datafusion] 01/01: remove reference to physical plan in logical plan
Posted by ji...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
jiayuliu pushed a commit to branch logical-plan
in repository https://gitbox.apache.org/repos/asf/arrow-datafusion.git
commit 0d891207cd35693cdaed905d9357a35bd26b9fa5
Author: Jiayu Liu <ji...@hey.com>
AuthorDate: Wed Feb 9 18:09:29 2022 +0800
remove reference to physical plan in logical plan
---
datafusion/src/logical_plan/expr.rs | 3 +--
datafusion/src/logical_plan/registry.rs | 6 ++----
2 files changed, 3 insertions(+), 6 deletions(-)
diff --git a/datafusion/src/logical_plan/expr.rs b/datafusion/src/logical_plan/expr.rs
index ba853ee..f075aaa 100644
--- a/datafusion/src/logical_plan/expr.rs
+++ b/datafusion/src/logical_plan/expr.rs
@@ -22,8 +22,6 @@ pub use super::Operator;
use crate::error::Result;
use crate::logical_plan::ExprSchemable;
use crate::logical_plan::{DFField, DFSchema};
-use crate::physical_plan::udaf::AggregateUDF;
-use crate::physical_plan::udf::ScalarUDF;
use arrow::datatypes::DataType;
use datafusion_common::DataFusionError;
pub use datafusion_common::{Column, ExprSchema};
@@ -33,6 +31,7 @@ use datafusion_expr::BuiltinScalarFunction;
pub use datafusion_expr::Expr;
use datafusion_expr::StateTypeFunction;
pub use datafusion_expr::{lit, lit_timestamp_nano, Literal};
+use datafusion_expr::{AggregateUDF, ScalarUDF};
use datafusion_expr::{
ReturnTypeFunction, ScalarFunctionImplementation, Signature, Volatility,
};
diff --git a/datafusion/src/logical_plan/registry.rs b/datafusion/src/logical_plan/registry.rs
index d9b1839..f439a12 100644
--- a/datafusion/src/logical_plan/registry.rs
+++ b/datafusion/src/logical_plan/registry.rs
@@ -15,11 +15,9 @@
// specific language governing permissions and limitations
// under the License.
-use std::{collections::HashSet, sync::Arc};
-
use crate::error::Result;
-use crate::physical_plan::udaf::AggregateUDF;
-use crate::physical_plan::udf::ScalarUDF;
+use datafusion_expr::{AggregateUDF, ScalarUDF};
+use std::{collections::HashSet, sync::Arc};
/// A registry knows how to build logical expressions out of user-defined function' names
pub trait FunctionRegistry {