darknight commented on PR #2004: URL: https://github.com/apache/incubator-opendal/pull/2004#issuecomment-1510377259
> > do you wish not have new API, instead resue existing interface? > > I prefer not to add new APIs. Ok, then perhaps I'll reuse `write_with(path: &str, args: OpWrite, bs: impl Into<Bytes>)` API in `Operator` instead of creating a new one. And in order to do that, I need to add one more flag field in `OpWrite` struct, to indicate whether current write is a append operation or normal write operation. This way I suppose we can avoid to make change on `Operator`, `Accessor` and all related `LayeredAccessor`s, sounds good? -- 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]
