Skip to content

Commit

Permalink
FIx
Browse files Browse the repository at this point in the history
Signed-off-by: Xuanwo <[email protected]>
  • Loading branch information
Xuanwo committed Mar 20, 2024
1 parent 5bb760c commit 7b9fb4a
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions core/src/docs/rfcs/4382_range_based_read.md
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@ However, after observing our users, we found that:

- `AsyncSeek` in `Reader` is prone to misuse.
- `Reader` does not support concurrent reading.
- `Reader` can't adopt Completion-based IO

## Misuse of `AsyncSeek`

Expand All @@ -43,6 +44,10 @@ For example, `s3` storage does not support `seek` natively. When users call `see

There is no way for opendal to support this feature.

## Can't adopt Completion-based IO

Completion-based IO requires take the buffer's owner ship. But API that take `&mut [u8]` can't do that.

# Guide-level explanation

So I propose to convert `Reader` into a stateless, range-based reading pattern.
Expand Down

0 comments on commit 7b9fb4a

Please sign in to comment.