etseidl commented on PR #7307: URL: https://github.com/apache/arrow-rs/pull/7307#issuecomment-3443712138
I agree that before putting too much effort into this PR we agree on the correct way to implement row numbers (I defer to @alamb and others for the arrow side of this). One concern I have with the approach here is how to provide exact row numbers if we start selectively reading row group metadata. If we don't have metadata for all preceding row groups, we can't know the starting row number. This at least argues for reverting back to using an `Option` for the start index. -- 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]
