iceberg-python
iceberg-python copied to clipboard
Deprecate `adlfs.*` configuration properties in favor of `adls.*`
Feature Request / Improvement
To set up a configuration to ADLS, we use now adlfs.* prefixed properties. This is refers to the implementation, and we want to use adls. See https://py.iceberg.apache.org/configuration/
What we want to do:
- Replace
adlfstoadls, and emit a warning when we see this to let the user know that aldfs is being deprecated. - Update the properties and docs with
adls.
Hello @Fokko,
I'd like to resolve this issue. I just want to confirm the expected behavior. Since we are replacing ADLFS with ADLS, I believe the expected behavior should be to raise an error. If we choose to warn instead, it means we are not replacing ADLFS but supporting both, and we are implying that we will eventually stop supporting the ADLFS prefix.
What should the expected behavior be?
closed by #961