Skip to content

Commit bf6a97a

Browse files
authored
make builder public under experimental (#7825)
Make `ArrayReaderBuilder` public (under experimental), I think the ability to build array reader was public prior to #7521 This will allow downstream users to build their own array readers. This is also consistent with many other array readers that are public.
1 parent a9f316b commit bf6a97a

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

parquet/src/arrow/array_reader/builder.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,12 +35,12 @@ use crate::errors::{ParquetError, Result};
3535
use crate::schema::types::{ColumnDescriptor, ColumnPath, Type};
3636

3737
/// Builds [`ArrayReader`]s from parquet schema, projection mask, and RowGroups reader
38-
pub(crate) struct ArrayReaderBuilder<'a> {
38+
pub struct ArrayReaderBuilder<'a> {
3939
row_groups: &'a dyn RowGroups,
4040
}
4141

4242
impl<'a> ArrayReaderBuilder<'a> {
43-
pub(crate) fn new(row_groups: &'a dyn RowGroups) -> Self {
43+
pub fn new(row_groups: &'a dyn RowGroups) -> Self {
4444
Self { row_groups }
4545
}
4646

parquet/src/arrow/array_reader/mod.rs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,8 @@ mod struct_array;
4545
#[cfg(test)]
4646
mod test_util;
4747

48-
pub(crate) use builder::ArrayReaderBuilder;
48+
// Note that this crate is public under the `experimental` feature flag.
49+
pub use builder::ArrayReaderBuilder;
4950
pub use byte_array::make_byte_array_reader;
5051
pub use byte_array_dictionary::make_byte_array_dictionary_reader;
5152
#[allow(unused_imports)] // Only used for benchmarks

0 commit comments

Comments
 (0)