Skip to content

Commit d68b800

Browse files
authored
Minor: Deprecate unused PartitionedFileStream (apache#20869)
## Which issue does this PR close? N/A ## Rationale for this change I ran across this unused type, so let's deprecate it so it is clear it is actually unused ## What changes are included in this PR? 1. Deprecate type ## Are these changes tested? By CI ## Are there any user-facing changes? <!-- If there are user-facing changes then we may require documentation to be updated before approving the PR. --> <!-- If there are any breaking changes to public APIs, please add the `api change` label. -->
1 parent 21cf60a commit d68b800

File tree

2 files changed

+8
-3
lines changed
  • datafusion

2 files changed

+8
-3
lines changed

datafusion/core/src/datasource/listing/mod.rs

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,8 @@
2121
mod table;
2222
pub use datafusion_catalog_listing::helpers;
2323
pub use datafusion_catalog_listing::{ListingOptions, ListingTable, ListingTableConfig};
24-
pub use datafusion_datasource::{
25-
FileRange, ListingTableUrl, PartitionedFile, PartitionedFileStream,
26-
};
24+
// Keep for backwards compatibility until removed
25+
#[expect(deprecated)]
26+
pub use datafusion_datasource::PartitionedFileStream;
27+
pub use datafusion_datasource::{FileRange, ListingTableUrl, PartitionedFile};
2728
pub use table::ListingTableConfigExt;

datafusion/datasource/src/mod.rs

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,10 @@ use std::pin::Pin;
7171
use std::sync::Arc;
7272

7373
/// Stream of files get listed from object store
74+
#[deprecated(
75+
since = "54.0.0",
76+
note = "This type is unused and will be removed in a future release"
77+
)]
7478
pub type PartitionedFileStream =
7579
Pin<Box<dyn Stream<Item = Result<PartitionedFile>> + Send + Sync + 'static>>;
7680

0 commit comments

Comments
 (0)