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

Skip re-reading entries that were just written #361

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mhutchinson
Copy link
Contributor

This addresses #360.

@mhutchinson mhutchinson requested a review from AlCutter November 14, 2024 14:35
@mhutchinson mhutchinson marked this pull request as ready for review November 18, 2024 11:13
@mhutchinson
Copy link
Contributor Author

Before:

❯ go run ./cmd/loadtest --log_count 3 --target=http://192.168.0.228:80/
...
I1113 17:28:56.461765 1334490 loadtest.go:124] Average update time is 582.347509ms with throttle Current max: 5/s. Oversupply in last second: 0
I1113 17:29:00.950903 1334490 loadtest.go:124] Average update time is 400.88653ms with throttle Current max: 6/s. Oversupply in last second: 0
I1113 17:29:04.472492 1334490 loadtest.go:124] Average update time is 397.475675ms with throttle Current max: 7/s. Oversupply in last second: 0
I1113 17:29:07.973485 1334490 loadtest.go:124] Average update time is 397.36765ms with throttle Current max: 8/s. Oversupply in last second: 0
I1113 17:29:11.404905 1334490 loadtest.go:124] Average update time is 414.572738ms with throttle Current max: 9/s. Oversupply in last second: 1
I1113 17:29:19.312549 1334490 loadtest.go:124] Average update time is 951.590867ms with throttle Current max: 9/s. Oversupply in last second: 1
I1113 17:29:22.650107 1334490 loadtest.go:124] Average update time is 403.399989ms with throttle Current max: 9/s. Oversupply in last second: 2
I1113 17:29:27.584686 1334490 loadtest.go:124] Average update time is 569.889193ms with throttle Current max: 9/s. Oversupply in last second: 6
I1113 17:29:33.384958 1334490 loadtest.go:124] Average update time is 721.465079ms with throttle Current max: 9/s. Oversupply in last second: 5
I1113 17:29:39.826264 1334490 loadtest.go:124] Average update time is 746.911001ms with throttle Current max: 9/s. Oversupply in last second: 5
I1113 17:29:53.837773 1334490 loadtest.go:124] Average update time is 1.688207382s with throttle Current max: 9/s. Oversupply in last second: 6
F1113 17:29:53.837793 1334490 loadtest.go:133] Witness took 1.688207382s on average to handle last 25 requests with throttle: Current max: 9/s. Oversupply in last second: 6

After:

❯ go run ./cmd/loadtest --log_count 3 --target=http://192.168.0.228:80/
...
I1118 11:27:28.182302 4081176 loadtest.go:125] Average update time is 276.024116ms with throttle Current max: 5/s. Oversupply in last second: 0
I1118 11:27:32.916400 4081176 loadtest.go:125] Average update time is 285.799367ms with throttle Current max: 6/s. Oversupply in last second: 0
I1118 11:27:36.834519 4081176 loadtest.go:125] Average update time is 422.529204ms with throttle Current max: 7/s. Oversupply in last second: 0
I1118 11:27:40.544102 4081176 loadtest.go:125] Average update time is 445.376604ms with throttle Current max: 8/s. Oversupply in last second: 0
I1118 11:27:44.117220 4081176 loadtest.go:125] Average update time is 430.781767ms with throttle Current max: 9/s. Oversupply in last second: 2
I1118 11:27:51.662743 4081176 loadtest.go:125] Average update time is 899.168464ms with throttle Current max: 9/s. Oversupply in last second: 0
I1118 11:27:54.062216 4081176 loadtest.go:125] Average update time is 295.786825ms with throttle Current max: 10/s. Oversupply in last second: 0
I1118 11:27:56.551938 4081176 loadtest.go:125] Average update time is 298.63643ms with throttle Current max: 11/s. Oversupply in last second: 1
I1118 11:27:59.112922 4081176 loadtest.go:125] Average update time is 297.619778ms with throttle Current max: 11/s. Oversupply in last second: 1
I1118 11:28:01.520360 4081176 loadtest.go:125] Average update time is 298.530196ms with throttle Current max: 11/s. Oversupply in last second: 2
I1118 11:28:04.086261 4081176 loadtest.go:125] Average update time is 297.964895ms with throttle Current max: 11/s. Oversupply in last second: 1
I1118 11:28:06.576410 4081176 loadtest.go:125] Average update time is 294.379389ms with throttle Current max: 11/s. Oversupply in last second: 0
I1118 11:28:09.139660 4081176 loadtest.go:125] Average update time is 305.434338ms with throttle Current max: 12/s. Oversupply in last second: 2
I1118 11:28:11.513089 4081176 loadtest.go:125] Average update time is 301.06091ms with throttle Current max: 12/s. Oversupply in last second: 1
I1118 11:28:14.005688 4081176 loadtest.go:125] Average update time is 299.106236ms with throttle Current max: 12/s. Oversupply in last second: 2
I1118 11:28:20.638780 4081176 loadtest.go:125] Average update time is 795.899539ms with throttle Current max: 12/s. Oversupply in last second: 11
I1118 11:28:24.394320 4081176 loadtest.go:125] Average update time is 450.592344ms with throttle Current max: 12/s. Oversupply in last second: 6
I1118 11:28:28.251925 4081176 loadtest.go:125] Average update time is 458.136641ms with throttle Current max: 12/s. Oversupply in last second: 6
I1118 11:28:31.846328 4081176 loadtest.go:125] Average update time is 432.640149ms with throttle Current max: 12/s. Oversupply in last second: 5
I1118 11:28:34.898879 4081176 loadtest.go:125] Average update time is 369.628665ms with throttle Current max: 12/s. Oversupply in last second: 5

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

Successfully merging this pull request may close these issues.

2 participants