Ted-Jiang commented on PR #1977:
URL: https://github.com/apache/arrow-rs/pull/1977#issuecomment-1172540557
> Edit: I'll try and stub out some APIs for what I mean over the next couple
of days. This will also help me validate my mental model checks out 😅
Got it, i will delete the `get_record_reader_by_columns_and_row_ranges` and
use options avoid public APIs
tomorrow i will start with
> * Add a skip page function to SerializedPageReader that uses the column
index to skip the next page without reading it (we may need to change it to
take ChunkReader instead of Read)
After read the code, i think if we want to skip page in
`SerializedPageReader` we need get the page meta, but in SerializedPageReader
it only care about the decode work. the pages offset already set to `buf: T` in
`SerializedRowGroupReader` so i try to pass column index to
`SerializedRowGroupReader` to change pages offset.
So one question 🤔
So if i want to add `skip page` i need add owned page location and
selected row to `SerializedPageReader`?
I got worried about where should i pass the column index info.
--
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]
