Mike Percy has submitted this change and it was merged.

Change subject: Improve tpch usability
......................................................................


Improve tpch usability

* tpch.sh should use $ROOT to find the CPU scaling script (I had a
  problem with it otherwise)
* tpch1.cc should print the error if the CHECK fails (e.g. cannot create
  the data directory)
* Allow for skipping the line count check (e.g. if we run with SF != 1)
* Allow for passing command-line parameters to tpch1

Change-Id: I604b0f7433781be78d76ca34f3e30e49ddeca6a8
Reviewed-on: http://gerrit.cloudera.org:8080/3279
Reviewed-by: Adar Dembo <a...@cloudera.com>
Reviewed-by: Jean-Daniel Cryans
Tested-by: Mike Percy <mpe...@apache.org>
---
M src/kudu/benchmarks/tpch/tpch1.cc
M src/kudu/scripts/tpch.sh
2 files changed, 12 insertions(+), 8 deletions(-)

Approvals:
  Jean-Daniel Cryans: Looks good to me, approved
  Mike Percy: Verified
  Adar Dembo: Looks good to me, approved



-- 
To view, visit http://gerrit.cloudera.org:8080/3279
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I604b0f7433781be78d76ca34f3e30e49ddeca6a8
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Mike Percy <mpe...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Jean-Daniel Cryans
Gerrit-Reviewer: Mike Percy <mpe...@apache.org>

Reply via email to