Riza Suminto has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22033 )

Change subject: IMPALA-13505: Fix NPE in Calcite Planner
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/22033/1/fe/src/main/java/org/apache/impala/analysis/Analyzer.java
File fe/src/main/java/org/apache/impala/analysis/Analyzer.java:

http://gerrit.cloudera.org:8080/#/c/22033/1/fe/src/main/java/org/apache/impala/analysis/Analyzer.java@4705
PS1, Line 4705: .getCanonicalPath()
              :               .subList(0, 3)
Is it a good place to address my previous comment?
https://gerrit.cloudera.org/c/21142/67/fe/src/main/java/org/apache/impala/analysis/Analyzer.java#4700



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I4d282120e596fd39a569d1ce9b25024f4f174dd0
Gerrit-Change-Number: 22033
Gerrit-PatchSet: 1
Gerrit-Owner: Jason Fehr <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
Gerrit-Comment-Date: Tue, 05 Nov 2024 21:26:52 +0000
Gerrit-HasComments: Yes

Reply via email to