tustvold commented on code in PR #3102:
URL: https://github.com/apache/arrow-rs/pull/3102#discussion_r1023393798
##########
parquet/src/file/reader.rs:
##########
@@ -143,6 +145,10 @@ pub trait RowGroupReader: Send + Sync {
Ok(col_reader)
}
+ #[cfg(feature = "bloom")]
Review Comment:
I'm suggesting rather than providing a lazy API to read the bloom filter on
demand, provide an API to make SerializedReader load blook filters as part of
ParquetMetadata if the corresponding feature and ReadOption is enabled.
This is necessary to be able to support object stores, and is generally a
good idea to avoid lots of small IO reads.
--
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]
