Skip to content

Make backend.getFactory public #213

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

Open
SQUIDwarrior opened this issue Sep 24, 2019 · 0 comments · May be fixed by #405
Open

Make backend.getFactory public #213

SQUIDwarrior opened this issue Sep 24, 2019 · 0 comments · May be fixed by #405

Comments

@SQUIDwarrior
Copy link
Contributor

SQUIDwarrior commented Sep 24, 2019

Is your feature request related to a problem? Please describe.
The backend.getFactory function should be made public. This can allow backends to be written can instantiate any other backend implementation (S3, HDFS, etc.) without having to re-implement the factory method explicitly. For example, I am working on a "shadow" client that can read/write to multiple backends and allow fallback from one to the other, say S3 and HDFS. However to implement this I have to re-implement the HDFS and S3 backend client factory methods since I can't use "getFactory" to access already registered factory functions.

Describe the solution you'd like
Make backend.getFactory a public method.

Describe alternatives you've considered
I have had to re-implement the factory methods for backend clients I am using in my "shadow" client

@hweawer hweawer linked a pull request May 8, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant