Impala Public Jenkins has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/11845 )

Change subject: IMPALA-5946,IMPALA-5956: add TPC-DS q31,q59,q89
......................................................................

IMPALA-5946,IMPALA-5956: add TPC-DS q31,q59,q89

Q31: the substitution variables didn't match the TPC-DS spec.
     After fixing this, the results match up to 4 digits of
     rounding (there is some error introduced in intermediate
     calculations).
Q59: the results match the reference results up to rounding.
Q89: the results match up to 5 digits of rounding.

I verified the matches by using a spreadsheet comparing reference
and actual results.
https://docs.google.com/spreadsheets/d/1MNEqkfYRRAd3xqY6m20tTHquqjtCThDaGdizzRAQ8co/edit?usp=sharing

*https://github.com/gregrahn/tpcds-kit/blob/master/specification/TPC-DS_v2.10.0.pdf
^https://github.com/gregrahn/tpcds-kit/tree/master/answer_sets

Change-Id: I49634e8f63066773c9c78dd5585a0ee69daf720a
Reviewed-on: http://gerrit.cloudera.org:8080/11845
Reviewed-by: Impala Public Jenkins <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
---
M testdata/workloads/tpcds-unmodified/queries/tpcds-q31.test
M testdata/workloads/tpcds-unmodified/queries/tpcds-q59.test
M testdata/workloads/tpcds-unmodified/queries/tpcds-q89.test
M tests/query_test/test_tpcds_queries.py
4 files changed, 358 insertions(+), 66 deletions(-)

Approvals:
  Impala Public Jenkins: Looks good to me, approved; Verified

--
To view, visit http://gerrit.cloudera.org:8080/11845
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I49634e8f63066773c9c78dd5585a0ee69daf720a
Gerrit-Change-Number: 11845
Gerrit-PatchSet: 3
Gerrit-Owner: Tim Armstrong <[email protected]>
Gerrit-Reviewer: David Knupp <[email protected]>
Gerrit-Reviewer: Greg Rahn <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>

Reply via email to