tustvold opened a new issue, #2990: URL: https://github.com/apache/arrow-datafusion/issues/2990
**Is your feature request related to a problem or challenge? Please describe what you are trying to do.** Prior to https://github.com/apache/arrow-datafusion/pull/2677/files#diff-0171c6cc11900557bf3bb9bd9fce2a78b8272e7f63ed8259c988ff610334da04L48 there was a trait called `FormatReaderOpener`. I replaced this with a trait called `FormatReader` but in hindsight this name is confusing for two reasons: * It suggests the trait is involved in reading the file, when in reality it is only responsible for creating the reader * It suggests an instance should exist for each format, when in reality they exist for each `FileStream`, i.e. partition **Describe the solution you'd like** I would like to rename `FormatReader` to `FileOpener` and `ReaderFuture` to `FileOpenFuture` **Describe alternatives you've considered** We could not do this **Additional context** This trait came up in discussion on https://github.com/apache/arrow-rs/issues/2230 and I realised its name was unhelpful -- 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]
