tilinna opened a new issue #12428:
URL: https://github.com/apache/arrow/issues/12428


   I didn't find any code from v8-master using the io.Reader interface of the 
`parquet.ReaderAtSeeker`. And the only usage for the io.Seeker was to find out 
the size of the file.
   
   ```go
   type ReaderAtSeeker interface {
        io.ReaderAt
        io.ReadSeeker
   }
   ```
   Therefore, I propose the interface to be simplified (and possibly renamed) 
to:
   
   ```go
   type SizedReaderAt interface {
        Size() int64
        io.ReaderAt
   }
   ```
   
   For example, io.SectionReader and bytes.Reader already implement this 
combination, so the changes should be minimal.
   
   Also, when converting a client-provided ReaderAtSeeker (_SizedReaderAt_) 
into a io.SectionReader (in buffered reading mode), it would be a good idea to 
first test whether the instance is already implementing the SectionReader API 
too.
   


-- 
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]


Reply via email to