Skip to content

Deprecate adlfs.* configuration properties in favor of adls.* #866

Closed

Description

@Fokko

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.

Activity

  1. ndrluis commented on Jul 4, 2024

    @ndrluis
    Collaborator

    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?

  2. self-assigned this
    on Aug 7, 2024
  3. kevinjqliu commented on Nov 15, 2024

    @kevinjqliu
    Contributor

    closed by #961

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions