Michael Smith has posted comments on this change. ( http://gerrit.cloudera.org:8080/22033 )
Change subject: IMPALA-13505: Fix NPE in Calcite Planner ...................................................................... Patch Set 2: Code-Review+1 (3 comments) 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@4698 PS1, Line 4698: return; // Note, since this statement is within a lambda function, this return > The Calcite planner produces this join node: Ack http://gerrit.cloudera.org:8080/#/c/22033/1/tests/custom_cluster/test_workload_mgmt_sql_details.py File tests/custom_cluster/test_workload_mgmt_sql_details.py: http://gerrit.cloudera.org:8080/#/c/22033/1/tests/custom_cluster/test_workload_mgmt_sql_details.py@461 PS1, Line 461: res = client.execute("SELECT s_store_name, sum(ss_net_profit) FROM store_sales," > I was able to eliminate most of the zip codes from the IN list, but that wa Ack http://gerrit.cloudera.org:8080/#/c/22033/1/tests/custom_cluster/test_workload_mgmt_sql_details.py@521 PS1, Line 521: > The Calcite planner does not set any of the workload management related fie Ack -- 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: 2 Gerrit-Owner: Jason Fehr <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jason Fehr <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Reviewer: Steve Carlin <[email protected]> Gerrit-Comment-Date: Wed, 06 Nov 2024 20:38:37 +0000 Gerrit-HasComments: Yes
