yugabyte / tpcc

Repo to run TPCC benchmarks against YugabyteDB
Other
24 stars 21 forks source link

Number of terminals might be wrong, when startFrom is used #141

Closed eivanov89 closed 11 months ago

eivanov89 commented 1 year ago

Hi folks,

Here you seem to want comparing agains numWarehouses + workConf.getStartWarehouseIdForShard() - 1.

sonalsagarwal commented 1 year ago

This part of code seems to be redundant, as we are computing and setting the value for upperTerminalId just before this 'if' condition.

hbhanawat commented 11 months ago

Closing as per the comment above.