Delivery transaction doesn't confirm TPC-C standard

[eivanov89]

Hi folks,

At @YDB we 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 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 must 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.

[apavlo]

@eivanov89 Can you send a PR with your fix?

[eivanov89]

@apavlo Sure, I will create the PR soon.