kevinjqliu commented on issue #744: URL: https://github.com/apache/arrow-rs-object-store/issues/744#issuecomment-4688257558
One idea from reviewing #724: since this is already a breaking-change release, should we remove `cloud`? Once we introduce `*-base`, `reqwest`, and explicit crypto provider features, `cloud` becomes less clear. #724 keeps `cloud` for backwards compatibility: ```toml # Compatibility/convenience alias. # Historically, cloud meant "common cloud support", including reqwest. # Keep that behavior for existing users. cloud = ["cloud-base", "reqwest"] ``` Given that `cloud` is just `cloud-base` + `reqwest`, it does not seem useful to keep in a breaking-change release. Users who want the default setup can enable `aws`, `azure`, `gcp`, or `http` directly. Users who want custom transport or crypto can use the corresponding `*-base` feature and compose the pieces they need. I would still keep `cloud-base`, though. It is still useful as the shared implementation feature that provider-specific base features compose from: -- 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]
