suyanhanx commented on PR #2036:
URL: 
https://github.com/apache/incubator-opendal/pull/2036#issuecomment-1514474440

   > No, we are confused many things together here.
   > 
   > After this PRs change, there is no `append` concept, we have two different 
write uses cases: Sized and Unsized.
   > 
   > If users specified the size for `write` operation and provide all data 
with it, we can write with oneshot. Otherwise, services need to handle it in 
different ways: append, multipart upload, range write, ...
   > 
   > `op.write()` is exactly this case: we know the size and we have all data. 
`op.writer()` is the other case: we don't know the size in advance, and 
services need to handle it by different ways.
   > 
   > In general, we will hide all the impelment details inside services. The 
only thing for users to use is `write`, and the only choice for users is 
specify the size or not.
   
   Great explanation! I think this needs to be in the documentation.


-- 
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]

Reply via email to