-
Notifications
You must be signed in to change notification settings - Fork 181
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
socket hang up #142
Comments
One thing I found is that |
I tried to add |
Digging deeper I found that getting all the results objects is what causes the slowness: |
If I limit the 'from' time to 3 days instead of 30, that helps significantly!
|
Hi Danny, ouch! Yeah I guess we're not really geared towards that many URLs quite yet. Your suggestion of adding references to the last run result in a task object might be sensible, but whatever happens Node.js is still loading a lot of data into memory. I'd like to be able to fix this, and it'll definitely inform development on Sidekick. |
Seems this is not only when you have that many URLs but also when you have a lot of "notices" or any kind of issues. For example I have 56 result objects, some of which have ~900 subresults (individual errors/warns/notices) and this is enough for the pa11y-dashboard homepage to timeout. I am surprised people are able to have the dashboard open with weeks worth of data... must not have a lot of accessibility errors. |
This issue seems to be plaguing me as well. Hopefully a fix can be found soon. |
I'm experiencing this error quite often. I'm not sure why. I do have over 4000 URLs imported, and noticed the dashboard slowed down significantly. Would a queue that size be too large? It was running fine on 800 URLs for a while.
The text was updated successfully, but these errors were encountered: