Xuanwo commented on issue #1130: URL: https://github.com/apache/iceberg-rust/issues/1130#issuecomment-2750166032
I see. We have three approachs so far: 1. Support `storage-webhdfs` and `storage-hdfs` but only exposes as `hdfs` to users Users are required to use feature flags for configuration. 2. Have only `storage-hdfs` but allow users to pick up impls like `webhdfs` or `hdfs` at runtime by config. Provides more control for users, but it may be a bit confusing when feature flags don't match the configuration. 3. Exposes `storage-webhdfs` and `storage-hdfs` seperately It doesn't work for webhdfs since all tables will use `hdfs://abc`. -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
