Refactor API into `into_futures_async_read_seek` and `into_futures_async_read`
- Dominant language
- Rust
- Stars
- 5.4k
- Forks
- 825
- Avg merge
- 1d 14m
- Merged PRs (30d)
- 127
Description
> Maybe there should be be two impls then. One which supports seek, and one that does not.
> We have no need for seek, for example. We just need the latest copy.
>
> Yep, that's a good idea. I think we can introduce an `into_futures_async_read_seek`. And that's will be a breaking change.
>
> Do you want to work on a fix on this?
_Originally posted by @Xuanwo in [#6995](https://github.com/apache/opendal/issues/6995#issuecomment-3646933231)_
Contributor guide
Research direction
Start with the existing API discussed in issue #6995 and trace where its async read and seek behavior is defined. Separate the interfaces into `into_futures_async_read_seek` and `into_futures_async_read`, then verify that the breaking API change is consistently reflected in the relevant callers and tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- backend-api-design
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100