javasoze commented on PR #5837: URL: https://github.com/apache/opendal/pull/5837#issuecomment-2740850273
> Thank you for this PR! As I mentioned in [this comment](https://github.com/apache/opendal/issues/5825#issuecomment-2738996065), we plan to introduce more options in the future (such as `if_match`), so we can't add the API in the way proposed in this PR. Instead, we might consider adding a `ReadOptions` struct. Thanks, @Xuanwo! This makes sense. reading/writing with offset/len is rather standard in most of the Java APIs that involve reading/writing. Do you think it makes sense to do something like: ```java byte[] read(String path, long offset, long len, ReadOptions options) ``` I am not sure `options` is a good place to put offset/len. As a client developer, constructing options (following the documentation) seems a little convoluted just to read a part of a file. Just my $0.02. -- 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: commits-unsubscr...@opendal.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
