Ji-Xinyou commented on PR #3017: URL: https://github.com/apache/incubator-opendal/pull/3017#issuecomment-1708437534
> > As some users need it. You can't remove it without giving an alternative. > > Makes sense, we can keep `sink` and `copy` API but changing its internal details. By saying keeping the API but changing the details. I think the current problem is about the argument type. 1. 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. 2. 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? -- 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]
