Joe McDonnell has uploaded this change for review. ( http://gerrit.cloudera.org:8080/22532
Change subject: IMPALA-13781: Fix "Median Diff %" for report_benchmark_results.py ...................................................................... IMPALA-13781: Fix "Median Diff %" for report_benchmark_results.py The calculation that report_benchmark_results.py uses for "Median Diff %" is using the wrong divisor. For an AB test, it should be dividing by the base median time (i.e. A), but this is actually dividing by the new median time (i.e. B). That gives the wrong range and can result in "Median Diff %" values of >100%. This fixes the calculation to divide by the base median time (i.e. A). Change-Id: I6f9da14bc7d5b6ef6a795b9024760f8cae77d72f --- M tests/benchmark/report_benchmark_results.py 1 file changed, 2 insertions(+), 2 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/32/22532/1 -- To view, visit http://gerrit.cloudera.org:8080/22532 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I6f9da14bc7d5b6ef6a795b9024760f8cae77d72f Gerrit-Change-Number: 22532 Gerrit-PatchSet: 1 Gerrit-Owner: Joe McDonnell <[email protected]>
