alamb commented on code in PR #7502:
URL: https://github.com/apache/arrow-rs/pull/7502#discussion_r2093304953
##########
parquet/src/arrow/arrow_reader/mod.rs:
##########
@@ -814,9 +807,10 @@ impl ParquetRecordBatchReader {
/// simplify error handling with `?`
fn next_inner(&mut self) -> Result<Option<RecordBatch>> {
let mut read_records = 0;
- match self.selection.as_mut() {
+ let batch_size = self.batch_size();
+ match self.read_plan.selection_mut() {
Review Comment:
The idea is in future PRs we will change this control loop so it can use
`BooleanArray` when that is a more efficient representation. Specifically, I
think we can follow the model @zhuqi-lucas did in
https://github.com/apache/arrow-rs/pull/7454
--
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: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
