Hello Zoltan Borok-Nagy, Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/22922

to look at the new patch set (#13).

Change subject: IMPALA-13625: Allow reading Parquet int32/int64 as decimal 
without logical types
......................................................................

IMPALA-13625: Allow reading Parquet int32/int64 as decimal without logical types

Modify ParquetMetadataUtils::ValidateColumn to allow reading int32 and
int64 as decimals if the difference of (precision - scale) is more than
9 and 18 respectively. After fulfilling the query a warning is still
raised that column does not have converted type set to DECIMAL.

Test:
* modify existing e2e test to accomodate the change
* add new e2e tests

Change-Id: I56006eb3cca28c81ec8467d77b35005fbf669680
---
M be/src/exec/parquet/parquet-data-converter.h
M be/src/exec/parquet/parquet-metadata-utils.cc
M 
testdata/workloads/functional-query/queries/QueryTest/parquet-type-widening.test
3 files changed, 123 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/22/22922/13
--
To view, visit http://gerrit.cloudera.org:8080/22922
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I56006eb3cca28c81ec8467d77b35005fbf669680
Gerrit-Change-Number: 22922
Gerrit-PatchSet: 13
Gerrit-Owner: Daniel Vanko <[email protected]>
Gerrit-Reviewer: Daniel Vanko <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>

Reply via email to