[Impala-ASF-CR] IMPALA-6105: Clarify argument order in single node perf run
Jim Apple has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8470 ) Change subject: IMPALA-6105: Clarify argument order in single_node_perf_run .. IMPALA-6105: Clarify argument order in single_node_perf_run single_node_perf_run.py uses git_hash_A vs. git_hash_B, distinguish them by their position in the command-line arguments. single_node_perf_run.py calls report_benchmark_results.py, which uses the "reference vs. input", distinguished by their command-line flags. The output of report_benchmark_results.py uses "{empty string} vs Base". In the long run, I think it would be better to fix all three to use the same terminology, but this comment hopefully adds clarity. Change-Id: Ib236ce7e83dc193ef1382f630ce58759a639 Reviewed-on: http://gerrit.cloudera.org:8080/8470 Tested-by: Impala Public Jenkins Reviewed-by: Jim Apple--- M bin/single_node_perf_run.py 1 file changed, 22 insertions(+), 2 deletions(-) Approvals: Impala Public Jenkins: Verified Jim Apple: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/8470 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ib236ce7e83dc193ef1382f630ce58759a639 Gerrit-Change-Number: 8470 Gerrit-PatchSet: 3 Gerrit-Owner: Jim Apple Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Jim Apple Gerrit-Reviewer: Tim Armstrong
[Impala-ASF-CR] IMPALA-6105: Clarify argument order in single node perf run
Jim Apple has posted comments on this change. ( http://gerrit.cloudera.org:8080/8470 ) Change subject: IMPALA-6105: Clarify argument order in single_node_perf_run .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8470 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib236ce7e83dc193ef1382f630ce58759a639 Gerrit-Change-Number: 8470 Gerrit-PatchSet: 2 Gerrit-Owner: Jim AppleGerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Jim Apple Gerrit-Reviewer: Tim Armstrong Gerrit-Comment-Date: Tue, 07 Nov 2017 16:16:07 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-6105: Clarify argument order in single node perf run
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8470 ) Change subject: IMPALA-6105: Clarify argument order in single_node_perf_run .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/8470 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib236ce7e83dc193ef1382f630ce58759a639 Gerrit-Change-Number: 8470 Gerrit-PatchSet: 2 Gerrit-Owner: Jim AppleGerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Tim Armstrong Gerrit-Comment-Date: Tue, 07 Nov 2017 08:08:59 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-6105: Clarify argument order in single node perf run
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8470 ) Change subject: IMPALA-6105: Clarify argument order in single_node_perf_run .. Patch Set 2: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1439/ -- To view, visit http://gerrit.cloudera.org:8080/8470 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib236ce7e83dc193ef1382f630ce58759a639 Gerrit-Change-Number: 8470 Gerrit-PatchSet: 2 Gerrit-Owner: Jim AppleGerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Tim Armstrong Gerrit-Comment-Date: Tue, 07 Nov 2017 04:59:01 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-6105: Clarify argument order in single node perf run
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8470 ) Change subject: IMPALA-6105: Clarify argument order in single_node_perf_run .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8470 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib236ce7e83dc193ef1382f630ce58759a639 Gerrit-Change-Number: 8470 Gerrit-PatchSet: 1 Gerrit-Owner: Jim AppleGerrit-Reviewer: Tim Armstrong Gerrit-Comment-Date: Mon, 06 Nov 2017 21:25:01 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-6105: Clarify argument order in single node perf run
Jim Apple has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8470 Change subject: IMPALA-6105: Clarify argument order in single_node_perf_run .. IMPALA-6105: Clarify argument order in single_node_perf_run single_node_perf_run.py uses git_hash_A vs. git_hash_B, distinguish them by their position in the command-line arguments. single_node_perf_run.py calls report_benchmark_results.py, which uses the "reference vs. input", distinguished by their command-line flags. The output of report_benchmark_results.py uses "{empty string} vs Base". In the long run, I think it would be better to fix all three to use the same terminology, but this comment hopefully adds clarity. Change-Id: Ib236ce7e83dc193ef1382f630ce58759a639 --- M bin/single_node_perf_run.py 1 file changed, 22 insertions(+), 2 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/70/8470/1 -- To view, visit http://gerrit.cloudera.org:8080/8470 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ib236ce7e83dc193ef1382f630ce58759a639 Gerrit-Change-Number: 8470 Gerrit-PatchSet: 1 Gerrit-Owner: Jim Apple