Skip to content
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

Aggregate finders #26

Closed
sagikazarmark opened this issue Mar 25, 2024 · 1 comment
Closed

Aggregate finders #26

sagikazarmark opened this issue Mar 25, 2024 · 1 comment

Comments

@sagikazarmark
Copy link
Owner

Currently there is no way to specify pairs of paths and names as a search criteria (ie. all names are looked up in all paths which may not be desirable).

One potential solution is eliminating the difference between paths and names.

Another solution is creating a separate finder implementation that aggregates different finders (maybe running them in parallel).

This may not result in the best performance, but the primary purpose of this library is to support configuration loading use cases where performance is rarely a concern.

Introducing alternate implementations comes with a new challenge though: we need a finder interface in this library. The name Finder is already taken though.

@sagikazarmark
Copy link
Owner Author

I decided to tackle this in spf13/viper#1849 instead.

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

No branches or pull requests

1 participant