iceberg-python icon indicating copy to clipboard operation
iceberg-python copied to clipboard

Deprecate `adlfs.*` configuration properties in favor of `adls.*`

Open Fokko opened this issue 1 year ago • 1 comments

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 adlfs to adls, 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.

Fokko avatar Jun 28 '24 09:06 Fokko

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?

ndrluis avatar Jul 04 '24 13:07 ndrluis

closed by #961

kevinjqliu avatar Nov 15 '24 00:11 kevinjqliu