This is an automated email from the ASF dual-hosted git repository.
xuanwo pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/iceberg-rust.git
The following commit(s) were added to refs/heads/main by this push:
new e519c3ae chore: Remove deprecated `remove_all` in FileIO (#1863)
e519c3ae is described below
commit e519c3aed12ec762b360e7ee9b53530b6214015e
Author: Jonathan Chen <[email protected]>
AuthorDate: Mon Nov 17 04:02:12 2025 -0500
chore: Remove deprecated `remove_all` in FileIO (#1863)
## Which issue does this PR close?
<!--
We generally require a GitHub issue to be filed for all bug fixes and
enhancements and this helps us generate change logs for our releases.
You can link an issue to this PR using the GitHub syntax. For example
`Closes #123` indicates that this PR will close issue #123.
-->
- Closes #.
## What changes are included in this PR?
<!--
Provide a summary of the modifications in this PR. List the main changes
such as new features, bug fixes, refactoring, or any other updates.
-->
## Are these changes tested?
<!--
Specify what test covers (unit test, integration test, etc.).
If tests are not included in your PR, please explain why (for example,
are they covered by existing tests)?
-->
---
crates/iceberg/src/io/file_io.rs | 12 ------------
1 file changed, 12 deletions(-)
diff --git a/crates/iceberg/src/io/file_io.rs b/crates/iceberg/src/io/file_io.rs
index 6e2d152e..24e91ca8 100644
--- a/crates/iceberg/src/io/file_io.rs
+++ b/crates/iceberg/src/io/file_io.rs
@@ -93,17 +93,6 @@ impl FileIO {
Ok(op.delete(relative_path).await?)
}
- /// Remove the path and all nested dirs and files recursively.
- ///
- /// # Arguments
- ///
- /// * path: It should be *absolute* path starting with scheme string used
to construct [`FileIO`].
- #[deprecated(note = "use remove_dir_all instead", since = "0.4.0")]
- pub async fn remove_all(&self, path: impl AsRef<str>) -> Result<()> {
- let (op, relative_path) = self.inner.create_operator(&path)?;
- Ok(op.remove_all(relative_path).await?)
- }
-
/// Remove the path and all nested dirs and files recursively.
///
/// # Arguments
@@ -292,7 +281,6 @@ pub struct FileMetadata {
/// Trait for reading file.
///
/// # TODO
-///
/// It's possible for us to remove the async_trait, but we need to figure
/// out how to handle the object safety.
#[async_trait::async_trait]