Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11805 )
Change subject: IMPALA-7761: Add multiple DISTINCT to targeted perf and stress test ...................................................................... IMPALA-7761: Add multiple DISTINCT to targeted perf and stress test IMPALA-110 added support for queries with multiple DISTINCT aggregates in a single select list. This patch adds queries to test this functionality to our targeted-perf workloads and fixes some incorrect return types in another targeted-perf aggregation query. It also adds some targeted queries to the stress test by extending the regex for stress test files to accept files of the form 'tpch-stress-*' and to allow for multiple tests per file. Testing: - Added an e2e test that runs the stress test file. Change-Id: I400aaf6b6620b4001895eafff785956bffb312c9 Reviewed-on: http://gerrit.cloudera.org:8080/11805 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M testdata/workloads/targeted-perf/queries/aggregation.test M testdata/workloads/tpcds/queries/tpcds-q39-1.test M testdata/workloads/tpcds/queries/tpcds-q39-2.test A testdata/workloads/tpch/queries/tpch-stress-aggregations.test M tests/infra/test_stress_infra.py M tests/query_test/test_aggregation.py M tests/util/parse_util.py M tests/util/test_file_parser.py 8 files changed, 112 insertions(+), 13 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/11805 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I400aaf6b6620b4001895eafff785956bffb312c9 Gerrit-Change-Number: 11805 Gerrit-PatchSet: 5 Gerrit-Owner: Thomas Marshall <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Michael Brown <[email protected]> Gerrit-Reviewer: Thomas Marshall <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
