rdettai commented on pull request #8300: URL: https://github.com/apache/arrow/pull/8300#issuecomment-708440082
An argument in favor of having `ChunkReader` be implemented for `T:ParquetReader` rather than `File`: -> this allows to write wrappers around `File` that you can then pass to `SerializedFileReader`. For instance at some points I wanted to debug the number of actual read ops on the `File` object so I wrote a tiny wrapper that incremented a counter at each `read()` call... ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
