Peter Rozsa has uploaded a new patch set (#3). ( 
http://gerrit.cloudera.org:8080/20197 )

Change subject: IMPALA-12285: Use targetType in string-to-numeric literal 
conversion
......................................................................

IMPALA-12285: Use targetType in string-to-numeric literal conversion

This change fixes mismatched type problems when an implicitly casted
string literal gets converted to a numeric type. Example:

'INSERT INTO example(float_col) VALUES ("0"), (15629);'

After this change, StringLiteral's 'convertToNumber' method will
consider the targetType parameter when creates a new NumericLiteral.

Test:
 - test case added to insert-unsafe.test

Change-Id: I2141e7ab164af55a7fa66dda05fe6dcbd7379b69
---
M fe/src/main/java/org/apache/impala/analysis/StringLiteral.java
M testdata/workloads/functional-query/queries/QueryTest/insert-unsafe.test
2 files changed, 8 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/97/20197/3
--
To view, visit http://gerrit.cloudera.org:8080/20197
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I2141e7ab164af55a7fa66dda05fe6dcbd7379b69
Gerrit-Change-Number: 20197
Gerrit-PatchSet: 3
Gerrit-Owner: Peter Rozsa <[email protected]>
Gerrit-Reviewer: Daniel Becker <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Peter Rozsa <[email protected]>

Reply via email to