Skip to content
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

Lazy load Script text #338

Merged
merged 2 commits into from Sep 12, 2018

Conversation

Projects
None yet
2 participants
@BlythMeister
Copy link
Contributor

BlythMeister commented Sep 10, 2018

Part 2 of #331
Also including Part 1 (#337)

Lazy load of scripts_run column

Stats

Current Master

Run1 Run 2 Run 3 Average
2:17.248 1:52.886 1:59.482 2:03.205

Part 1 (#337)

Run1 Run 2 Run 3 Average
1.34.084 1:29.291 1:38.848 1:34.074

This PR

Run1 Run 2 Run 3 Average
1:31.176 1:31.206 1:29.419 1:30.600

BlythMeister added some commits Sep 10, 2018

@BiggerNoise BiggerNoise merged commit b57291b into chucknorris:master Sep 12, 2018

2 checks passed

Codacy/PR Quality Review Up to standards. A positive pull request.
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details

@BiggerNoise BiggerNoise referenced this pull request Sep 12, 2018

Closed

ScriptRunCache Map #340

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.