anujphadke has posted comments on this change. Change subject: IMPALA-5145: Do not constant fold null in CastExprs ......................................................................
Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/6663/3/fe/src/main/java/org/apache/impala/rewrite/FoldConstantsRule.java File fe/src/main/java/org/apache/impala/rewrite/FoldConstantsRule.java: Line 55: // cast-to-types and that can lead to query failures, e.x., CTAS > e.g. e.g. http://gerrit.cloudera.org:8080/#/c/6663/1/testdata/workloads/functional-query/queries/QueryTest/create-table-as-select.test File testdata/workloads/functional-query/queries/QueryTest/create-table-as-select.test: Line 187: # IMPALA-5145: Do not constant fold null in CastExprs > Yes, Moving the test here and it does not catch the issue(tried it with and Yes, Moved* the test here and it does not catch the issue(tried it with and without the new change). -- To view, visit http://gerrit.cloudera.org:8080/6663 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia7aa1ab7f53a9dcc7560ded321a9d1e1ee2d18e3 Gerrit-PatchSet: 3 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: anujphadke <[email protected]> Gerrit-Reviewer: Alex Behm <[email protected]> Gerrit-Reviewer: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: anujphadke <[email protected]> Gerrit-HasComments: Yes
