hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "pengcheng xiong" <pxi...@hortonworks.com>
Subject Review Request 38136: CBO: Calcite Operator To Hive Operator (Calcite Return Path): ctas after order by has problem
Date Fri, 04 Sep 2015 22:12:27 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38136/
-----------------------------------------------------------

Review request for hive and John Pullokkaran.


Repository: hive-git


Description
-------

with return path on, "create table b as select * from src order by key" will fail. Attached
two test cases, one (cbo_rp_cross_product_check_2.q) is to test that "create table b as select
* from src order by key" will not fail. The other(cbo_rp_auto_join17.q) is to correct current
wrong table alias on master.


Diffs
-----

  ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/translator/PlanModifierForASTConv.java
5cd3a06 
  ql/src/java/org/apache/hadoop/hive/ql/parse/CalcitePlanner.java 73ae7c4 
  ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 778c7b2 
  ql/src/test/queries/clientpositive/cbo_rp_auto_join17.q PRE-CREATION 
  ql/src/test/queries/clientpositive/cbo_rp_cross_product_check_2.q PRE-CREATION 
  ql/src/test/results/clientpositive/cbo_rp_auto_join17.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/cbo_rp_cross_product_check_2.q.out PRE-CREATION 

Diff: https://reviews.apache.org/r/38136/diff/


Testing
-------


Thanks,

pengcheng xiong


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message