Qifan Chen has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/18050 )

Change subject: [WIP] IMPALA-10992 Planner changes for estimate peak memory - v2
......................................................................


Patch Set 19:

Resolve these issues:


P22

Query:                                                                          
           
delete from functional_kudu.testtbl                                     
Error Stack:                                           
java.lang.IllegalStateException   
  at com.google.common.base.Preconditions.checkState(Preconditions.java:492)    
          == resolved by asking DeleteStmt copy cstr to call the full version 
of ModifyStmt's
  at org.apache.impala.analysis.DeleteStmt.createDataSink(DeleteStmt.java:59)
  at org.apache.impala.planner.Planner.createPlanFragments(Planner.java:203)  
  at org.apache.impala.planner.Planner.createPlans(Planner.java:305)
  at org.apache.impala.service.Frontend.createExecRequest(Frontend.java:1562)
  at 
org.apache.impala.service.Frontend.getPlannedExecRequest(Frontend.java:1939)
  at org.apache.impala.service.Frontend.doCreateExecRequest(Frontend.java:1779)
  at org.apache.impala.service.Frontend.getTExecRequest(Frontend.java:1644)     
  at org.apache.impala.service.Frontend.createExecRequest(Frontend.java:1614)
  at 
org.apache.impala.planner.PlannerTestBase.testPlan(PlannerTestBase.java:513)
  at 
org.apache.impala.planner.PlannerTestBase.runTestCase(PlannerTestBase.java:424)
  at 
org.apache.impala.planner.PlannerTestBase.runPlannerTestFile(PlannerTestBase.java:909)
  at 
org.apache.impala.planner.PlannerTestBase.runPlannerTestFile(PlannerTestBase.java:935)
  at org.apache.impala.planner.PlannerTest.testKuduDelete(PlannerTest.java:685)
  at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)

P23

insert into functional_kudu.testtbl(id, name)
select count(distinct id), name from functional_kudu.dimtbl                     
    
group by name                                                  
Error Stack:                              
java.lang.IllegalStateException                                                 
            
  at com.google.common.base.Preconditions.checkState(Preconditions.java:492)    
       <== resolved by copying partitionColPos_ in InsertStmt.java
  at 
org.apache.impala.analysis.KuduPartitionExpr.<init>(KuduPartitionExpr.java:46)
  at org.apache.impala.util.KuduUtil.createPartitionExpr(KuduUtil.java:478)   
  at 
org.apache.impala.planner.DistributedPlanner.createInsertFragment(DistributedPlanner.java:288)
  at org.apache.impala.planner.Planner.createPlanFragments(Planner.java:183)    
   
  at org.apache.impala.planner.Planner.createPlans(Planner.java:303)
  at org.apache.impala.service.Frontend.createExecRequest(Frontend.java:1562)   
            
  at 
org.apache.impala.service.Frontend.getPlannedExecRequest(Frontend.java:1939)
  at org.apache.impala.service.Frontend.doCreateExecRequest(Frontend.java:1779)
  at org.apache.impala.service.Frontend.getTExecRequest(Frontend.java:1644)
  at org.apache.impala.service.Frontend.createExecRequest(Frontend.java:1614)
  at 
org.apache.impala.planner.PlannerTestBase.testPlan(PlannerTestBase.java:513)
  at 
org.apache.impala.planner.PlannerTestBase.runTestCase(PlannerTestBase.java:424)
  at 
org.apache.impala.planner.PlannerTestBase.runPlannerTestFile(PlannerTestBase.java:909)
  at 
org.apache.impala.planner.PlannerTestBase.runPlannerTestFile(PlannerTestBase.java:845)
  at org.apache.impala.planner.PlannerTest.testKudu(PlannerTest.java:668)
  at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)        
  at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)

P24:

update functional_kudu.testtbl set name = 'peter' where zip > 94549             
             
Error Stack:                                                            
java.lang.IllegalStateException                
  at com.google.common.base.Preconditions.checkState(Preconditions.java:492)    
     <===     resolved by asking UpdateStmt copy cstr to call the full version 
of ModifyStmt's
  at org.apache.impala.analysis.UpdateStmt.createDataSink(UpdateStmt.java:67)
  at org.apache.impala.planner.Planner.createPlanFragments(Planner.java:197)   
  at org.apache.impala.planner.Planner.createPlans(Planner.java:303)
  at org.apache.impala.service.Frontend.createExecRequest(Frontend.java:1562)   
        
  at 
org.apache.impala.service.Frontend.getPlannedExecRequest(Frontend.java:1939)
  at org.apache.impala.service.Frontend.doCreateExecRequest(Frontend.java:1779)
  at org.apache.impala.service.Frontend.getTExecRequest(Frontend.java:1644)


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If8a31a574b364f39b049a4bae33a8b98c5fc20bd
Gerrit-Change-Number: 18050
Gerrit-PatchSet: 19
Gerrit-Owner: Qifan Chen <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Kurt Deschler <[email protected]>
Gerrit-Reviewer: Qifan Chen <[email protected]>
Gerrit-Reviewer: Wenzhe Zhou <[email protected]>
Gerrit-Comment-Date: Wed, 15 Dec 2021 18:10:15 +0000
Gerrit-HasComments: No

Reply via email to