Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11087 )
Change subject: IMPALA-7354: planner test resource estimates for more workloads ...................................................................... IMPALA-7354: planner test resource estimates for more workloads Adds the resource estimates for key benchmark workloads: TPC-H, TPC-DS, TPC-H Nested and TPC-H Kudu to the planner test so that we can track changes in resource requirements and estimates for these queries. Also don't show decimal places for MB and KB estimates. The estimates are not accurate to that level and displaying extra precision has some disadvantages: * It communicates to readers that the estimates have a high level of precision. * It increases the odds of small variations in file sizes, etc causing test failures. Also fixed a regex in the stress test that didn't escape the decimal point correctly. Testing: Ran core tests. Change-Id: I6a9f836699200ea87fb03bf36abad0e23949ac26 Reviewed-on: http://gerrit.cloudera.org:8080/11087 Reviewed-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> --- M fe/src/main/java/org/apache/impala/common/PrintUtils.java M fe/src/main/java/org/apache/impala/planner/Planner.java M fe/src/test/java/org/apache/impala/planner/PlannerTest.java M fe/src/test/java/org/apache/impala/planner/PlannerTestBase.java A fe/src/test/java/org/apache/impala/util/PrintUtilsTest.java M testdata/workloads/functional-planner/queries/PlannerTest/disable-codegen.test M testdata/workloads/functional-planner/queries/PlannerTest/max-row-size.test M testdata/workloads/functional-planner/queries/PlannerTest/resource-requirements.test M testdata/workloads/functional-planner/queries/PlannerTest/spillable-buffer-sizing.test M testdata/workloads/functional-planner/queries/PlannerTest/tpcds-all.test M testdata/workloads/functional-planner/queries/PlannerTest/tpch-all.test M testdata/workloads/functional-planner/queries/PlannerTest/tpch-kudu.test M testdata/workloads/functional-planner/queries/PlannerTest/tpch-nested.test M testdata/workloads/functional-query/queries/QueryTest/admission-reject-mem-estimate.test M testdata/workloads/functional-query/queries/QueryTest/stats-extrapolation.test M tests/stress/concurrent_select.py 16 files changed, 925 insertions(+), 392 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/11087 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I6a9f836699200ea87fb03bf36abad0e23949ac26 Gerrit-Change-Number: 11087 Gerrit-PatchSet: 8 Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Reviewer: Bikramjeet Vig <bikramjeet....@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Reviewer: Vuk Ercegovac <vercego...@cloudera.com>
