Tim Armstrong has posted comments on this change. Change subject: IMPALA-3439: Only convert decimal literals in convertNumericLiteralsFromDecimal(). ......................................................................
Patch Set 1: I'm not sure that convertNumericLiteralsFromDecimal() makes sense in general, but I'm also not sure we can fix it without breaking compatibility. See IMPALA-3437 Does this patch affect the results of either of the examples in IMPALA-3437? -- To view, visit http://gerrit.cloudera.org:8080/3138 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I33f319b00a8fef3a81418d6c7ac030b002c5d0d1 Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: cdh5-2.6.0_5.8.0 Gerrit-Owner: Alex Behm <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-HasComments: No
