-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support registering new backends #45
Comments
Having looked at the code more thoroughly, I think we should:
Apart from now being able to register additional stores, this should also clean up FYI: @SimonBohnenQC |
This concept sounds much better than the current structure 👍🏼 As authentication was an issue in #51, I would suggest testing |
We would probably not be using |
I would love to keep |
I see the point of creating stores only from What do you mean by "schema validation" in this context? |
With "schema validation", I mean that you can specify a type schema for JSON/YAML files and validate that they have the correct values before creating the stores themselves. This has been useful in the past in using the stores in some settings to check whether the configuration is valid before deploying it to production. |
As a user of the
minimalkv
framework, I want to create and register new backends and use them through theget_store_from_url
function without changing theminimalkv
library. This is currently not possible, as theextract_params
function hard-codes the known storage types:https://github.com/data-engineering-collective/minimalkv/blob/main/minimalkv/_urls.py#L70-L122
What I'm imagining is a registration function that makes
minimalkv
aware of this new storage type, similarly to howfsspec
does it.The text was updated successfully, but these errors were encountered: