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

MultiPath creates non-specific path_id #92

Open
jnspitale opened this issue Aug 16, 2023 · 0 comments
Open

MultiPath creates non-specific path_id #92

jnspitale opened this issue Aug 16, 2023 · 0 comments

Comments

@jnspitale
Copy link
Collaborator

When '+' is supplied as the path_id, MultiPath registers the supplied paths using the ID of the first path with a '+' appended. This path_id could describe any number of possible collections of paths, and caused a conflict in the metadata code. The immediate solution was to not cache the paths. That solution is fine, but the conflict took time to track down, and could do so again in the future if not fixed.

My proposed solution, implemented in ideas/multipath/multipath.py, is for the path_id to be comprised of the names of all of the included paths in alphabetical order. Alphabetization is not necessary, but it helps to ensure that an identical group of paths are not cached more than once if they are specified in different orders.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants