Hello Thomas Marshall, David Knupp, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/12578
to look at the new patch set (#3).
Change subject: [stress] pull out QueryRunner
......................................................................
[stress] pull out QueryRunner
Further refactoring to reduce the size of concurrent_select.py.
Pull out the QueryRunner class and miscellaneous utility functions.
Improve encapsulation of _metrics by adding accessor functions.
Testing:
Ran locally with various arguments including DML and random queries.
Made it sure did some binary search by deleting parts of runtime info.
Change-Id: I69c907a65147d082211837cbbaba7225aa4b67cb
---
M tests/stress/concurrent_select.py
A tests/stress/query_runner.py
A tests/stress/util.py
3 files changed, 538 insertions(+), 469 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/78/12578/3
--
To view, visit http://gerrit.cloudera.org:8080/12578
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I69c907a65147d082211837cbbaba7225aa4b67cb
Gerrit-Change-Number: 12578
Gerrit-PatchSet: 3
Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-Reviewer: David Knupp <dkn...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Thomas Marshall <tmarsh...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
