alamb commented on code in PR #8733:
URL: https://github.com/apache/arrow-rs/pull/8733#discussion_r2511754194
##########
parquet/src/arrow/arrow_reader/read_plan.rs:
##########
@@ -233,12 +283,12 @@ pub struct ReadPlan {
/// The number of rows to read in each batch
batch_size: usize,
/// Row ranges to be selected from the data source
- selection: Option<VecDeque<RowSelector>>,
+ selection: Option<RowSelectionCursor>,
}
impl ReadPlan {
/// Returns a mutable reference to the selection, if any
- pub fn selection_mut(&mut self) -> Option<&mut VecDeque<RowSelector>> {
+ pub fn selection_mut(&mut self) -> Option<&mut RowSelectionCursor> {
Review Comment:
I propose this change in
- https://github.com/hhhizzz/arrow-rs/pull/8
--
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]
