alamb commented on code in PR #5774:
URL: https://github.com/apache/arrow-rs/pull/5774#discussion_r1603211150
##########
parquet/src/arrow/arrow_reader/mod.rs:
##########
@@ -44,12 +44,12 @@ use crate::file::page_index::index_reader;
pub use filter::{ArrowPredicate, ArrowPredicateFn, RowFilter};
pub use selection::{RowSelection, RowSelector};
-/// A generic builder for constructing sync or async arrow parquet readers.
This is not intended
-/// to be used directly, instead you should use the specialization for the
type of reader
-/// you wish to use
+/// Builder for constructing parquet readers into arrow.
///
-/// * For a synchronous API - [`ParquetRecordBatchReaderBuilder`]
-/// * For an asynchronous API - [`ParquetRecordBatchStreamBuilder`]
+/// Most users should use one of the following specializations:
+///
+/// * synchronous API: [`ParquetRecordBatchReaderBuilder::try_new`]
Review Comment:
the examples / APIs are on `try_new` so I think that is the place to direct
people rather than to `ParquetRecordBatchReaderBuilder` and then they have to
find try_new
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
