Daniel Becker has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16208 )

Change subject: IMPALA-7655: Implement codegen for conditional functions (if, 
isnull, coalesce)
......................................................................


Patch Set 1:

We have a number of cases of IF and COALESCE in 
testdata/workloads/functional-query/queries/QueryTest/ and we have the backend 
expr tests. I verified that the backend tests fail if I implement the 
conditional functions incorrectly.

Do you think we should run an exhaustive test?


--
To view, visit http://gerrit.cloudera.org:8080/16208
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I11f617a9148492ccafb46112ce0af103a10090f8
Gerrit-Change-Number: 16208
Gerrit-PatchSet: 1
Gerrit-Owner: Daniel Becker <[email protected]>
Gerrit-Reviewer: Bikramjeet Vig <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Daniel Becker <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Comment-Date: Fri, 17 Jul 2020 09:26:50 +0000
Gerrit-HasComments: No

Reply via email to