Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11315 )
Change subject: IMPALA-6923: Remove create_database.py and perf_result_datastore.py ...................................................................... IMPALA-6923: Remove create_database.py and perf_result_datastore.py Description: The current upstream usage indicates that report_benchmark_results.py script is the only script in this folder that is used to generate a report comparing performance between two given runs of the performance tests. The other two scripts create_database.py and perf_result_datastore.py store some metrics from a performance test run to database on a specified impala instance. However they rely on some Internal resources not available to external apache community to generate a meaningful interpretation/report of the metrics stored in the performance database. This change also removes lines of code from report_benchmark_results.py that were previously parsing command line options, that were being used by the two scripts that are being removed. Change-Id: I2c9b04f0d0e8ca221e383aab2ba3ee53d968dd10 Reviewed-on: http://gerrit.cloudera.org:8080/11315 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- D tests/benchmark/create_database.py D tests/benchmark/perf_result_datastore.py M tests/benchmark/report_benchmark_results.py 3 files changed, 15 insertions(+), 417 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/11315 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I2c9b04f0d0e8ca221e383aab2ba3ee53d968dd10 Gerrit-Change-Number: 11315 Gerrit-PatchSet: 3 Gerrit-Owner: Nithya Janarthanan <[email protected]> Gerrit-Reviewer: David Knupp <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jim Apple <[email protected]> Gerrit-Reviewer: Michael Brown <[email protected]>
