ByteStream: add support for from impl Read + Seek

[i18nsite]

Describe the feature

ByteStream: add support for impl Read

Use Case

I can pipeline from one reader to s3

Proposed Solution

No response

Other Information

No response

Acknowledgements

• I may be able to implement this feature request
• This feature might incur a breaking change

A note for the community

Community Note

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
• Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
• If you are interested in working on this issue, please leave a comment

[aajtodd]

Thanks for the feature request. ByteStream is modeled as an async API and trying to provide a sync Read implementation is not something we plan to support. We can't support Seek because not all streams are seekable.

You can turn a ByteStream into tokio::AsyncRead (see docs). You can then maybe leverage a crate from the ecosystem to bridge the sync and async worlds (example).

I'm going to go ahead and close this as we don't have plans to support this directly on ByteStream.

[github-actions]

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.