From 890e22580f500892ac81a605facca2cbc8ad7431 Mon Sep 17 00:00:00 2001 From: Piotr Findeisen Date: Mon, 4 Nov 2024 12:54:39 +0100 Subject: [PATCH 1/2] Remove deprecated transform functions They were deprecated since 38.0.0, which was released 6 months ago. --- datafusion/common/src/tree_node.rs | 18 ------------------ 1 file changed, 18 deletions(-) diff --git a/datafusion/common/src/tree_node.rs b/datafusion/common/src/tree_node.rs index 563f1fa85614..a0ad1e80be9b 100644 --- a/datafusion/common/src/tree_node.rs +++ b/datafusion/common/src/tree_node.rs @@ -238,15 +238,6 @@ pub trait TreeNode: Sized { transform_down_impl(self, &mut f) } - /// Same as [`Self::transform_down`] but with a mutable closure. - #[deprecated(since = "38.0.0", note = "Use `transform_down` instead")] - fn transform_down_mut Result>>( - self, - f: &mut F, - ) -> Result> { - self.transform_down(f) - } - /// Recursively rewrite the node using `f` in a bottom-up (post-order) /// fashion. /// @@ -271,15 +262,6 @@ pub trait TreeNode: Sized { transform_up_impl(self, &mut f) } - /// Same as [`Self::transform_up`] but with a mutable closure. - #[deprecated(since = "38.0.0", note = "Use `transform_up` instead")] - fn transform_up_mut Result>>( - self, - f: &mut F, - ) -> Result> { - self.transform_up(f) - } - /// Transforms the node using `f_down` while traversing the tree top-down /// (pre-order), and using `f_up` while traversing the tree bottom-up /// (post-order). From 94601f6d88bb3b76969ce1f19072cf33e2e69ac9 Mon Sep 17 00:00:00 2001 From: Piotr Findeisen Date: Mon, 4 Nov 2024 12:56:08 +0100 Subject: [PATCH 2/2] Remove deprecated and unused FileSinkExec type It was deprecated since 38.0.0, which was released 6 months ago. --- datafusion/physical-plan/src/insert.rs | 3 --- 1 file changed, 3 deletions(-) diff --git a/datafusion/physical-plan/src/insert.rs b/datafusion/physical-plan/src/insert.rs index e478cecb7ffc..ae8a2acce696 100644 --- a/datafusion/physical-plan/src/insert.rs +++ b/datafusion/physical-plan/src/insert.rs @@ -73,9 +73,6 @@ pub trait DataSink: DisplayAs + Debug + Send + Sync { ) -> Result; } -#[deprecated(since = "38.0.0", note = "Use [`DataSinkExec`] instead")] -pub type FileSinkExec = DataSinkExec; - /// Execution plan for writing record batches to a [`DataSink`] /// /// Returns a single row with the number of values written