-
Notifications
You must be signed in to change notification settings - Fork 106
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
Benchmarks #37
Comments
Also, it'd be nice to have a semi-recent benchmark in the readme. |
We have it in production and it consistently takes ~30-40ms on a t3.micro AWS instance. |
We also ran into this. It is by far the most expensive thing in serving a response for our service. Unfortunately, I don't think improving the situation would be easy as most of the time seems to be spent in |
@oschwald, I was able to drastically reduce the processing by caching the initialized parser. |
Thanks for the suggestion. We were already caching the parser. I ended up
switching us to github.com/avct/uasurfer, which has much better performance.
|
@Valve how do you cache the initialized parser? Can you pls some code snippets. |
#71 should be a pretty big win. You can switch to my fork if you'd like. |
#75 has another sped up version via caching. |
In #26, it shows that each op takes ~26ms. Is that still accurate?
The text was updated successfully, but these errors were encountered: