-
Notifications
You must be signed in to change notification settings - Fork 111
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
HTTP query queue speedup #1501
HTTP query queue speedup #1501
Conversation
|
Updated queue size to 20. Also if it is https with a non localhost hostname then it is set to 200. |
@magorlick would you mind taking a look at how this feels? This PR will hopefully make modeling feel a bit smoother / faster. |
try { | ||
if ( | ||
window.location.protocol === "https:" && | ||
window.location.host !== "localhost" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should this be https OR localhost instead of AND ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated
@AdityaHegde Here's what I'm seeing in this branch:
Kapture.2022-12-21.at.14.57.10.mp4 |
@hamilton Fixed them. Please check now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
* Fixing 500 errors * Improving for quick requests * Disabling tests and increasing queue size * Fixing timestamp profile and histogram priotiy * Allow navigation while porifiling * QueueSize = 200 for https or not localhost
No description provided.