Xuanwo commented on PR #3017: URL: https://github.com/apache/incubator-opendal/pull/3017#issuecomment-1708448423
> * If we are keeping the API, and only changing the details. How can we address the issue that we can not use `impl oio:Read` in the argument. > * If we are not keeping the nature of passing the stream, I think maybe we can deprecate the API in 0.40.0, because the `sink` is all about passing the stream right? There are some misunderstanding. I'm talking about - Keeping `opendal::Writer::sink()` and `opendal::Writer::copy()` - Removing `opendal::raw::oio::Write::copy_from()` There will not have argument issues anymore. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
