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

Read default font only once #106

Merged
merged 3 commits into from
Aug 13, 2022
Merged

Conversation

MarkEWaite
Copy link
Contributor

@MarkEWaite MarkEWaite commented Aug 13, 2022

Read the default font only once

  • Add more StatusImage tests to assure compatibility
  • Read default font only once for faster performance

Read default font only once for faster performance

JMeter test runs indicate that a single initialization of the default font reduces the HTTP response time by more than 50% and increases the throughout by more than 2x.

JMeter test runs were not exhaustive (30000 samples using a Linux controller running in a Docker container with the JMeter test from my Windows desktop), but they showed similar or better results on multiple runs.

Average response time before the change was 15ms. Average response time after the change ranged from 3-8 ms.

Median response time before the change was 10 ms. Median response time after the change was 2-3 ms with one 7ms outlier.

Throughput before the change was less than 200 requests per second. Throughput after the change was over 450 requests per second.

JMeter Aggregate Report Before

The JMeter aggregate test report before the change looked like this:

aggregate-before

JMeter Aggregate Report After

The JMeter aggregate test report after the change looked like this:

aggregate-after

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests - that demonstrates feature works or fixes the issue

JMeter test runs indicate that a single initialization of the
default font reduces the HTTP response time by more than 50% and
increases the throughout by more than 2x.

JMeter test runs were not exhaustive (30000 samples using a Linux
controller running in a Docker container with the JMeter test from
my Windows desktop), but they showed similar or better results on
multiple runs.

Average response time before the change was 15ms.
Average response time after the change ranged from 3-8 ms.

Median response time before the change was 10 ms.
Median response time after the change was 2-3 ms with one 7ms outlier.

Throughput before the change was less than 200 requests per second.
Throughput after the change was over 450 requests per second.
@MarkEWaite MarkEWaite added the enhancement Improvement or new feature label Aug 13, 2022
@MarkEWaite MarkEWaite merged commit a93c1c6 into jenkinsci:master Aug 13, 2022
@MarkEWaite MarkEWaite deleted the read-font-once branch August 13, 2022 19:38
@MarkEWaite
Copy link
Contributor Author

MarkEWaite commented Aug 15, 2022

See the JMeter test plan if the testing ever needs to be repeated. A second JMeter test plan may also be useful.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improvement or new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant