Taras Bobrovytsky has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/9062


Change subject: IMPALA-4924: Enable Decimal V2 by default
......................................................................

IMPALA-4924: Enable Decimal V2 by default

In this commit we enable Decimal_V2 by default. We also update the
expected results in many of our tests. We also remove the Decimal_V2
TPCDS workload. This patch should only be included in the Impala 3.0
branch.

Testing:
Ran an exhaustive test which almost passed. Updated the few failed
tests in it.

Change-Id: Ibbdd05bf986b7947f106b396017faa3a0bd87fd7
---
M be/src/exprs/expr-test.cc
M common/thrift/ImpalaInternalService.thrift
M fe/src/test/java/org/apache/impala/analysis/AnalyzeExprsTest.java
M fe/src/test/java/org/apache/impala/analysis/AnalyzeStmtsTest.java
M testdata/workloads/functional-query/queries/QueryTest/decimal-exprs.test
M testdata/workloads/functional-query/queries/QueryTest/decimal.test
M testdata/workloads/functional-query/queries/QueryTest/decimal_avro.test
M testdata/workloads/functional-query/queries/QueryTest/exprs.test
M testdata/workloads/functional-query/queries/QueryTest/java-udf.test
M 
testdata/workloads/functional-query/queries/QueryTest/libs_with_same_filenames.test
M testdata/workloads/functional-query/queries/QueryTest/load-java-udfs.test
M 
testdata/workloads/functional-query/queries/QueryTest/nested-types-subplan.test
M testdata/workloads/functional-query/queries/QueryTest/spilling-aggs.test
M testdata/workloads/functional-query/queries/QueryTest/uda.test
M 
testdata/workloads/functional-query/queries/QueryTest/udf-codegen-required.test
M testdata/workloads/functional-query/queries/QueryTest/udf-init-close.test
M 
testdata/workloads/functional-query/queries/QueryTest/udf-non-deterministic.test
M testdata/workloads/functional-query/queries/QueryTest/udf.test
M 
testdata/workloads/functional-query/queries/QueryTest/utc-timestamp-functions.test
M testdata/workloads/functional-query/queries/QueryTest/values.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q1.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q10a.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q11.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q12.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q13.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q15.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q16.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q17.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q18a.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q19.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q2.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q20.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q21.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q25.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q29.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q3.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q32.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q33.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q34.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q37.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q39-1.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q39-2.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q4.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q40.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q41.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q42.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q43.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q46.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q50.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q51.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q51a.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q52.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q53.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q54.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q55.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q56.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q6.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q60.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q61.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q62.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q64.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q65.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q67a.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q68.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q69.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q7.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q70a.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q71.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q72.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q73.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q74.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q75.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q76.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q77a.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q78.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q79.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q8.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q80a.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q81.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q82.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q84.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q86a.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q88.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q91.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q92.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q94.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q95.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q96.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q97.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q98.test
D testdata/workloads/tpcds/queries/tpcds-decimal_v2-q99.test
M testdata/workloads/tpcds/queries/tpcds-q1.test
M testdata/workloads/tpcds/queries/tpcds-q10a.test
M testdata/workloads/tpcds/queries/tpcds-q11.test
M testdata/workloads/tpcds/queries/tpcds-q12.test
M testdata/workloads/tpcds/queries/tpcds-q13.test
M testdata/workloads/tpcds/queries/tpcds-q15.test
M testdata/workloads/tpcds/queries/tpcds-q16.test
M testdata/workloads/tpcds/queries/tpcds-q17.test
M testdata/workloads/tpcds/queries/tpcds-q18a.test
M testdata/workloads/tpcds/queries/tpcds-q19.test
M testdata/workloads/tpcds/queries/tpcds-q2.test
M testdata/workloads/tpcds/queries/tpcds-q20.test
M testdata/workloads/tpcds/queries/tpcds-q21.test
R testdata/workloads/tpcds/queries/tpcds-q22a.test
M testdata/workloads/tpcds/queries/tpcds-q25.test
M testdata/workloads/tpcds/queries/tpcds-q29.test
M testdata/workloads/tpcds/queries/tpcds-q32.test
M testdata/workloads/tpcds/queries/tpcds-q33.test
M testdata/workloads/tpcds/queries/tpcds-q37.test
M testdata/workloads/tpcds/queries/tpcds-q39-1.test
M testdata/workloads/tpcds/queries/tpcds-q39-2.test
M testdata/workloads/tpcds/queries/tpcds-q40.test
M testdata/workloads/tpcds/queries/tpcds-q41.test
M testdata/workloads/tpcds/queries/tpcds-q50.test
M testdata/workloads/tpcds/queries/tpcds-q51.test
M testdata/workloads/tpcds/queries/tpcds-q51a.test
M testdata/workloads/tpcds/queries/tpcds-q53.test
M testdata/workloads/tpcds/queries/tpcds-q54.test
M testdata/workloads/tpcds/queries/tpcds-q56.test
M testdata/workloads/tpcds/queries/tpcds-q6.test
M testdata/workloads/tpcds/queries/tpcds-q60.test
M testdata/workloads/tpcds/queries/tpcds-q61.test
M testdata/workloads/tpcds/queries/tpcds-q62.test
M testdata/workloads/tpcds/queries/tpcds-q64.test
M testdata/workloads/tpcds/queries/tpcds-q67a.test
M testdata/workloads/tpcds/queries/tpcds-q69.test
M testdata/workloads/tpcds/queries/tpcds-q7.test
M testdata/workloads/tpcds/queries/tpcds-q70a.test
M testdata/workloads/tpcds/queries/tpcds-q71.test
M testdata/workloads/tpcds/queries/tpcds-q72.test
M testdata/workloads/tpcds/queries/tpcds-q74.test
M testdata/workloads/tpcds/queries/tpcds-q75.test
M testdata/workloads/tpcds/queries/tpcds-q76.test
M testdata/workloads/tpcds/queries/tpcds-q77a.test
M testdata/workloads/tpcds/queries/tpcds-q78.test
M testdata/workloads/tpcds/queries/tpcds-q80a.test
M testdata/workloads/tpcds/queries/tpcds-q81.test
M testdata/workloads/tpcds/queries/tpcds-q82.test
M testdata/workloads/tpcds/queries/tpcds-q84.test
M testdata/workloads/tpcds/queries/tpcds-q86a.test
M testdata/workloads/tpcds/queries/tpcds-q88.test
M testdata/workloads/tpcds/queries/tpcds-q91.test
M testdata/workloads/tpcds/queries/tpcds-q92.test
M testdata/workloads/tpcds/queries/tpcds-q94.test
M testdata/workloads/tpcds/queries/tpcds-q95.test
M testdata/workloads/tpcds/queries/tpcds-q97.test
M testdata/workloads/tpcds/queries/tpcds-q98.test
M testdata/workloads/tpcds/queries/tpcds-q99.test
M testdata/workloads/tpch/queries/tpch-q1.test
M testdata/workloads/tpch/queries/tpch-q14.test
M testdata/workloads/tpch/queries/tpch-q15.test
M testdata/workloads/tpch/queries/tpch-q17.test
M testdata/workloads/tpch/queries/tpch-q8.test
M testdata/workloads/tpch_nested/queries/tpch_nested-q1.test
M testdata/workloads/tpch_nested/queries/tpch_nested-q10.test
M testdata/workloads/tpch_nested/queries/tpch_nested-q11.test
M testdata/workloads/tpch_nested/queries/tpch_nested-q12.test
M testdata/workloads/tpch_nested/queries/tpch_nested-q13.test
M testdata/workloads/tpch_nested/queries/tpch_nested-q14.test
M testdata/workloads/tpch_nested/queries/tpch_nested-q15.test
M testdata/workloads/tpch_nested/queries/tpch_nested-q16.test
M testdata/workloads/tpch_nested/queries/tpch_nested-q17.test
M testdata/workloads/tpch_nested/queries/tpch_nested-q18.test
M testdata/workloads/tpch_nested/queries/tpch_nested-q19.test
M testdata/workloads/tpch_nested/queries/tpch_nested-q2.test
M testdata/workloads/tpch_nested/queries/tpch_nested-q20.test
M testdata/workloads/tpch_nested/queries/tpch_nested-q21.test
M testdata/workloads/tpch_nested/queries/tpch_nested-q22.test
M testdata/workloads/tpch_nested/queries/tpch_nested-q3.test
M testdata/workloads/tpch_nested/queries/tpch_nested-q4.test
M testdata/workloads/tpch_nested/queries/tpch_nested-q5.test
M testdata/workloads/tpch_nested/queries/tpch_nested-q6.test
M testdata/workloads/tpch_nested/queries/tpch_nested-q7.test
M testdata/workloads/tpch_nested/queries/tpch_nested-q8.test
M testdata/workloads/tpch_nested/queries/tpch_nested-q9.test
M tests/common/impala_test_suite.py
M tests/hs2/test_hs2.py
M tests/query_test/test_aggregation.py
M tests/query_test/test_decimal_casting.py
M tests/query_test/test_tpcds_queries.py
M tests/util/test_file_parser.py
182 files changed, 3,598 insertions(+), 17,737 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/62/9062/1
--
To view, visit http://gerrit.cloudera.org:8080/9062
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibbdd05bf986b7947f106b396017faa3a0bd87fd7
Gerrit-Change-Number: 9062
Gerrit-PatchSet: 1
Gerrit-Owner: Taras Bobrovytsky <tbobrovyt...@cloudera.com>

Reply via email to