Quanlong Huang has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/20274 )

Change subject: IMPALA-12319: Fix NullPointerException in 
UnnestExpr.isBoundByTupleIds()
......................................................................


Patch Set 3:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/20274/2/testdata/workloads/functional-planner/queries/PlannerTest/zipping-unnest.test
File 
testdata/workloads/functional-planner/queries/PlannerTest/zipping-unnest.test:

http://gerrit.cloudera.org:8080/#/c/20274/2/testdata/workloads/functional-planner/queries/PlannerTest/zipping-unnest.test@296
PS2, Line 296: UNNEST()
> It's strange the parameter for unnest is not visible in the output. Having
Nice catch! I think it's an issue of how we pass the labels of UnnestExpr and 
how we implement toSql() for it. Filed IMPALA-12324 to track it separatedly.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I43e1ef0467edfb8a4a23047a439426053c93ad72
Gerrit-Change-Number: 20274
Gerrit-PatchSet: 3
Gerrit-Owner: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Daniel Becker <[email protected]>
Gerrit-Reviewer: Gabor Kaszab <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Comment-Date: Sun, 30 Jul 2023 12:59:50 +0000
Gerrit-HasComments: Yes

Reply via email to