search

phetsims / aqua

Automatic QUality Assurance
MIT License
2 stars 4 forks source link

Improve ct-main server #169

Closed zepumph closed 1 year ago

zepumph commented 1 year ago

Discovered over in https://github.com/phetsims/special-ops/issues/234, the machine is now able to support 100 chrome clients pinging it, but we found some requests taking over 1.5 seconds. This is for the /test-result endpoint. @jonathanolson wonders if it has to do with saving the state to a file around 137 in ContinuousServer.js.

zepumph commented 1 year ago

the saveToFile() command was taking a long long time, and so we moved it to a separate loop instead of holding up responses. We may want to spend a bit more timing logic on other loops.

zepumph commented 1 year ago

This is good for now, and CT is really quite speedy. Ready to close