Skip to content

Commit 5f55674

Browse files
committed
fix merge conflict
1 parent 98cf8e2 commit 5f55674

File tree

4 files changed

+6
-7
lines changed

4 files changed

+6
-7
lines changed

datafusion/core/src/physical_planner.rs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,6 @@ use datafusion_physical_plan::unnest::ListUnnest;
9696
use sqlparser::ast::NullTreatment;
9797

9898
use async_trait::async_trait;
99-
use datafusion_datasource::file_groups::FileGroup;
10099
use datafusion_physical_plan::async_func::{AsyncFuncExec, AsyncMapper};
101100
use futures::{StreamExt, TryStreamExt};
102101
use itertools::{multiunzip, Itertools};

datafusion/expr/src/async_udf.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717

1818
use crate::{ReturnFieldArgs, ScalarFunctionArgs, ScalarUDF, ScalarUDFImpl};
1919
use arrow::array::ArrayRef;
20-
use arrow::datatypes::{DataType, Field, SchemaRef};
20+
use arrow::datatypes::{DataType, Field, FieldRef, SchemaRef};
2121
use async_trait::async_trait;
2222
use datafusion_common::config::ConfigOptions;
2323
use datafusion_common::error::Result;
@@ -52,15 +52,15 @@ pub trait AsyncScalarUDFImpl: Debug + Send + Sync {
5252
///
5353
/// By default, this function calls [`Self::return_type`] with the
5454
/// types of each argument.
55-
fn return_field_from_args(&self, args: ReturnFieldArgs) -> Result<Field> {
55+
fn return_field_from_args(&self, args: ReturnFieldArgs) -> Result<FieldRef> {
5656
let data_types = args
5757
.arg_fields
5858
.iter()
5959
.map(|f| f.data_type())
6060
.cloned()
6161
.collect::<Vec<_>>();
6262
let return_type = self.return_type(&data_types)?;
63-
Ok(Field::new(self.name(), return_type, true))
63+
Ok(Arc::new(Field::new(self.name(), return_type, true)))
6464
}
6565

6666
/// The ideal batch size for this function.
@@ -131,7 +131,7 @@ impl ScalarUDFImpl for AsyncScalarUDF {
131131
self.inner.return_type(arg_types)
132132
}
133133

134-
fn return_field_from_args(&self, args: ReturnFieldArgs) -> Result<Field> {
134+
fn return_field_from_args(&self, args: ReturnFieldArgs) -> Result<FieldRef> {
135135
self.inner.return_field_from_args(args)
136136
}
137137

datafusion/physical-plan/src/async_func.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ impl AsyncFuncExec {
7373
.iter()
7474
.map(|expr| (Arc::clone(&expr.func), expr.name().to_string()))
7575
.collect::<Vec<_>>();
76-
let async_expr_mapping = ProjectionMapping::try_new(&tuples, &input.schema())?;
76+
let async_expr_mapping = ProjectionMapping::try_new(tuples, &input.schema())?;
7777
let cache =
7878
AsyncFuncExec::compute_properties(&input, schema, &async_expr_mapping)?;
7979
Ok(Self {

datafusion/physical-plan/src/lib.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,8 +59,8 @@ mod visitor;
5959

6060
pub mod aggregates;
6161
pub mod analyze;
62-
pub mod coalesce;
6362
pub mod async_func;
63+
pub mod coalesce;
6464
pub mod coalesce_batches;
6565
pub mod coalesce_partitions;
6666
pub mod common;

0 commit comments

Comments
 (0)