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

Disable metapaths with path count too high #79

Open
vincerubinetti opened this issue May 30, 2019 · 1 comment
Open

Disable metapaths with path count too high #79

vincerubinetti opened this issue May 30, 2019 · 1 comment
Labels
enhancement New feature or request

Comments

@vincerubinetti
Copy link
Collaborator

As discussed here, querying for the list of paths for a metapath that has a very large number of paths (say 100,000) tends to crash the database and not return a result. Therefore, it seems like we'll have to limit which metapaths can be checked.

The tentative limit discussed is 10,000 paths. Any metapath with more than that many paths should be disabled from being checked, or should at least show a warning next to it.

@vincerubinetti vincerubinetti added the todo An action item label May 30, 2019
@vincerubinetti
Copy link
Collaborator Author

vincerubinetti commented Jul 30, 2019

I think we don't necessarily have to do this anymore, since this frontend is more robust and can't hang/crash as easily. The worst that happens is that the paths simply don't show up in the paths table (but the metapath stays checked).

If we were to do it though, I think the limit could be 100,000. I'm now fairly easily able to query paths < 100,000:

image

@vincerubinetti vincerubinetti added enhancement New feature or request and removed todo An action item labels Jul 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant