apache / apache/opendal

Refactor API into `into_futures_async_read_seek` and `into_futures_async_read`

Open
#7,003 1 comment 1 reaction 0 assignees View on GitHub
releases-note/refactor
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.