darknight commented on PR #2004: URL: https://github.com/apache/incubator-opendal/pull/2004#issuecomment-1510627798
> > hmm...the `append` field in `OpWrite` is for normal writing in multipart way (fetch `upload_id` if it's `true`), but the `append` we're talking about here is a separate operation, > > This's true, service may have different ways to implement current `Writer`'s API. > > > and I need another flag to differentiate `append write` from `multipart write`. > > However, this way violates our [VISION](https://opendal.apache.org/docs/vision) which makes users have to know whether a service supports `multipart` or `append`. > > We just need to choose a method for implementing the `append` semantics provided by the `Writer`. In the future, we can offer different methods for users to select through the `WasabiBuilder`. Well, if we don't need support `append write` and `multipart write` at the same time, and leave out compatibility with S3 service, then I suppose the implementation is much easier and no new flag field is needed. -- 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]
