Skip to content

Commit d7d86d9

Browse files
committed
make DistributedCodec not encode udfs
1 parent 23d640a commit d7d86d9

File tree

1 file changed

+14
-1
lines changed

1 file changed

+14
-1
lines changed

src/protobuf/distributed_codec.rs

Lines changed: 14 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,10 @@ use crate::{NetworkShuffleExec, PartitionIsolatorExec};
66
use bytes::Bytes;
77
use datafusion::arrow::datatypes::Schema;
88
use datafusion::arrow::datatypes::SchemaRef;
9-
use datafusion::common::internal_datafusion_err;
9+
use datafusion::common::{internal_datafusion_err, not_impl_err};
1010
use datafusion::error::DataFusionError;
1111
use datafusion::execution::FunctionRegistry;
12+
use datafusion::logical_expr::{AggregateUDF, ScalarUDF, WindowUDF};
1213
use datafusion::physical_expr::EquivalenceProperties;
1314
use datafusion::physical_plan::execution_plan::{Boundedness, EmissionType};
1415
use datafusion::physical_plan::{ExecutionPlan, Partitioning, PlanProperties};
@@ -228,6 +229,18 @@ impl PhysicalExtensionCodec for DistributedCodec {
228229
Err(proto_error(format!("Unexpected plan {}", node.name())))
229230
}
230231
}
232+
233+
fn try_encode_udf(&self, _: &ScalarUDF, _: &mut Vec<u8>) -> datafusion::common::Result<()> {
234+
not_impl_err!("DistributedCodec does not encode UDFs")
235+
}
236+
237+
fn try_encode_udaf(&self, _: &AggregateUDF, _: &mut Vec<u8>) -> datafusion::common::Result<()> {
238+
not_impl_err!("DistributedCodec does not encode UDAFs")
239+
}
240+
241+
fn try_encode_udwf(&self, _: &WindowUDF, _: &mut Vec<u8>) -> datafusion::common::Result<()> {
242+
not_impl_err!("DistributedCodec does not encode UDWFs")
243+
}
231244
}
232245

233246
/// A key that uniquely identifies a stage in a query

0 commit comments

Comments
 (0)