Skip to content

Commit

Permalink
Remove elements deprecated since v 38. (#13245)
Browse files Browse the repository at this point in the history
* Remove deprecated transform functions

They were deprecated since 38.0.0, which was released 6 months ago.

* Remove deprecated and unused FileSinkExec type

It was deprecated since 38.0.0, which was released 6 months ago.
  • Loading branch information
findepi authored Nov 4, 2024
1 parent 49b9351 commit 659d5a3
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 21 deletions.
18 changes: 0 additions & 18 deletions datafusion/common/src/tree_node.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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<F: FnMut(Self) -> Result<Transformed<Self>>>(
self,
f: &mut F,
) -> Result<Transformed<Self>> {
self.transform_down(f)
}

/// Recursively rewrite the node using `f` in a bottom-up (post-order)
/// fashion.
///
Expand All @@ -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<F: FnMut(Self) -> Result<Transformed<Self>>>(
self,
f: &mut F,
) -> Result<Transformed<Self>> {
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).
Expand Down
3 changes: 0 additions & 3 deletions datafusion/physical-plan/src/insert.rs
Original file line number Diff line number Diff line change
Expand Up @@ -73,9 +73,6 @@ pub trait DataSink: DisplayAs + Debug + Send + Sync {
) -> Result<u64>;
}

#[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
Expand Down

0 comments on commit 659d5a3

Please sign in to comment.