Adding throughput test to benchto #68
Conversation
|
|
|
As per TPC-H specification, we have to run power test and throughput test. currently benchto doesn't support throughput test and this PR is for to add that support into benchto. Original PR: trinodb/benchto#26 Co-authored-by: Paweł Pałucha, Karol Sobczak |
There was a problem hiding this comment.
@upbram123 Hi Ram, the back ported PR needs to follow the PR/commit message guidelines and include cherry pick PR/commit ids: https://github.com/prestodb/presto/wiki/Review-and-Commit-guidelines. Will you please update them? Thank you!
|
As per TPC-H specification, we have to run power test and throughput test. currently benchto doesn't support throughput test and this PR is for to add that support into benchto. Original PR: trinodb/benchto#26 Co-authored-by: Paweł Pałucha, Karol Sobczak |
As per TPC-H specification, we have to run power test and throughput test. currently benchto doesn't support throughput test and this PR is for to add that support into benchto. Original PR: trinodb/benchto#26 Cherry-pick of trinodb/benchto@a26b7f8 Co-authored-by: Paweł Pałucha, Karol Sobczak Adding throughput test to benchto
upbram123
force-pushed
the
throughput-test
branch
from
2d6163c
to
e671639
Compare
| int finalThread = thread; | ||
| executionCallables.add(() -> { | ||
| LOG.info("Running throughput test: {} queries, {} runs", benchmark.getQueries().size(), runs); | ||
| int[] queryOrder = PermutationUtils.preparePermutation(benchmark.getQueries().size(), finalThread); |
There was a problem hiding this comment.
We need to be able to run pre-defined ordered queries.
As per TPC-H specification, we have to run power test and throughput test. currently benchto doesn't support throughput test and this PR is for to add that support into benchto.
Original PR: trinodb/benchto#26
Cherry-pick of trinodb/benchto@a26b7f8
Co-authored-by: Paweł Pałucha, Karol Sobczak