Skip to content

Remove configured_scheme from OpenDalStorage::{S3, Azdls} #2245

@CTTY

Description

@CTTY

Is your feature request related to a problem or challenge?

Since now Storage API accepts fully-qualified locations, we no longer need to check schemes in OpenDalStorage::{S3, Azdls}

Describe the solution you'd like

  1. We should remove configured_scheme from OpenDalStorage::{S3, Azdls}
  2. In OpenDalResolvingStorage, we maintain a map<Scheme, Storage>, and since now we can have multiple schemes mapped to one storage instance: ("s3", "s3a", "s3n" -> S3Storage), we need to update this structure accordingly

for more context: #2231 (comment)

Willingness to contribute

None

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions