Skip to content

Results formatting: explicitly mark throughput as throughput; introduce goodput. #102

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

Merged
merged 2 commits into from
Jan 1, 2022

Conversation

lmwnshn
Copy link
Collaborator

@lmwnshn lmwnshn commented Dec 31, 2021

Explicitly mark throughput as throughput; introduce goodput.

This clarifies that the former requests/second number reported is a true
measure of the requests per second, i.e., it includes rejected and aborted
transactions.

Specifically, the former results string

Results(nanoSeconds=XXX, measuredRequests=XXX) = XXX.XXX requests/sec

is now of the form

Results(nanoSeconds=XXX, measuredRequests=XXX) = XXX.XXX requests/sec (throughput), XXX.XXX requests/sec (goodput)

where goodput is reported as the number of completed transactions.

We also write an additional Goodput line to the summary file generated.

This clarifies that the former requests/second number reported is a true
measure of the requests per second, i.e., it includes rejected and aborted
transactions.

Specifically, the former results string
Results(nanoSeconds=XXX, measuredRequests=XXX) = XXX.XXX requests/sec
is now of the form
Results(nanoSeconds=XXX, measuredRequests=XXX) = XXX.XXX requests/sec (throughput), XXX.XXX requests/sec (goodput)
where goodput is reported as the number of completed transactions.

We also write an additional Goodput line to the summary file generated.
@lmwnshn lmwnshn self-assigned this Dec 31, 2021
@lmwnshn lmwnshn merged commit fd4a69c into cmu-db:main Jan 1, 2022
@lmwnshn lmwnshn deleted the goodput branch January 1, 2022 03:18
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.

1 participant