jpopesculian commented on PR #7613:
URL: https://github.com/apache/arrow-rs/pull/7613#issuecomment-2944942408
Let me know how y'all feel about this. I think ideally actually we would get
rid of `AsyncFileWriter` and just use `AsyncWrite` directly. And for
`AsyncFileReader` use something like the following
```rust
trait AsyncFileReader: AsyncRead + AsyncSeek {
fn poll_metadata(
self: Pin<&mut Self>,
cx: &mut Context<'_>,
options: Option<&'a ArrowReaderOptions>
) -> Poll<Result<Arc<ParquetMetaData>>> {
// default implementation
}
}
```
and then implement the appropriate futures. That way we don't need to worry
about the specifics of Send and things
--
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]
