mixermt commented on PR #3111:
URL: https://github.com/apache/iceberg-rust/pull/3111#issuecomment-5811974531
Thanks @comphead. This is already an optional feature: `opendal-hdfs-native`
on `iceberg-storage-opendal`. It is off by default (default is memory, fs, and
s3) and not part of `opendal-all`. The backend is `cfg`-gated, and
`services-hdfs-native` is only a dependency when the feature is on, so a normal
Iceberg build does not compile it. A user opts in with:
```toml
iceberg-storage-opendal = { version = "...", features =
["opendal-hdfs-native"] }
```
If the goal is stronger than that — keeping `hdfs-native` out of this repo's
`Cargo.lock`, or out of `--all-features` CI — a feature cannot do it, because
the workspace lockfile and `--all-features` cover every feature. That would
mean a separate crate this workspace does not depend on. I would rather not
restructure unless that is what you want.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
