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

Delivery transaction doesn't confirm TPC-C standard #140

Open
eivanov89 opened this issue Aug 30, 2023 · 0 comments
Open

Delivery transaction doesn't confirm TPC-C standard #140

eivanov89 opened this issue Aug 30, 2023 · 0 comments

Comments

@eivanov89
Copy link

eivanov89 commented Aug 30, 2023

Hi folks,

At @YDB we also forked benchbase to implement TPC-C. Now we both share same bugs :)

Here you always iterate from 1 to terminalDistrictUpperID. It seems that original approach was either to iterate from terminalDistrictLowerID (left unused) to terminalDistrictUpperID. It means that first terminal iterates districts from 1 to 2, next terminal - 1 to 3, ..., 1 to 10. But each terminal should iterate 1 to 10 according the standard (and I double checked TPC-C implementation in CockroachDB).

At YDB bugfix decreased tpmc by 4%. In my opinion, this bug is critical, because all TPC-C results published are not accurate.

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

No branches or pull requests

1 participant