Steve Carlin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/25004 )

Change subject: IMPALA-15467: Calcite planner: Fix DECIMAL return type 
inference for IF
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/25004/2//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/25004/2//COMMIT_MSG@15
PS2, Line 15: Full upstream precommit and E2E are pending.
This change looks simple enough and I'll +1 it...but the e2e is pending?  I'd 
like to see (or know) a test that runs this on both original and Calcite 
planners.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3101917e02cae76c48c9d9aaf0d2913de00daffb
Gerrit-Change-Number: 25004
Gerrit-PatchSet: 2
Gerrit-Owner: Aleksandr Efimov <[email protected]>
Gerrit-Reviewer: Aleksandr Efimov <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Steve Carlin <[email protected]>
Gerrit-Comment-Date: Mon, 05 Oct 2026 22:19:42 +0000
Gerrit-HasComments: Yes

Reply via email to