Xuanwo commented on issue #5672: URL: https://github.com/apache/opendal/issues/5672#issuecomment-2692018921
Hi, before attempting to add a new feature to OpenDAL, it's highly recommended to first review OpenDAL's vision: https://opendal.apache.org/vision/. That's said that we must add new features in a way that aligns with out vision: **One Layer, All Storage.** We should do our best to avoid exposing any storage service-specific details in our public API, such as in `Operator` or `OpCopy`. That's why I believe adding `bucket` and `region` to `OpCopy` is not a good idea. These are configuration values specific to some storage services and are not relevant to other services. Additionally, this approach lacks extensibility, as it should also accommodate `SSE` settings effectively. One possible idea is adding new config values in `CosConfig` like `copy_source: Box<CosConfig>`, but I'm not sure if it's good engouth. -- 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]
