-
Notifications
You must be signed in to change notification settings - Fork 2
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
Network overview for large networks fails with 500 error #86
Comments
Are you sure the problem is due to too much ram usage? How did you find that out? Also, I noticed this section of documentation from that parameter:
We're currently using this function in full contrast of this statement: This would be for long running queries, and we currently do make use of |
I'm not sure that's the right option to use. I was thinking that the lock might only exist for queries that are mutating the data, not just reading. This would take a few minutes more research Here's the stack trace that shows it will use too much memory:
|
On a large network, the AQL command that would be executed takes up a lot of RAM and the command stops instead of going over the memory limit. This leads to silent failure for the user. There is an option to stream results at the link below, which might be useful.
https://docs.python-arango.com/en/main/specs.html#arango.aql.AQL.execute
We'd want to pass stream in this line (if it's a good solution): https://github.com/multinet-app/multinet-api/blob/main/multinet/api/views/common.py#L37
The text was updated successfully, but these errors were encountered: