You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Deprecate adlfs.* configuration properties in favor of adls.* #866
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.
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.
Activity
ndrluis commentedon Jul 4, 2024
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?
kevinjqliu commentedon Nov 15, 2024
closed by #961