hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ashutosh Chauhan <hashut...@apache.org>
Subject Review Request 54032: Enable CBO on interval literals
Date Wed, 23 Nov 2016 15:50:06 GMT

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

Review request for hive and pengcheng xiong.


Bugs: HIVE-15258
    https://issues.apache.org/jira/browse/HIVE-15258


Repository: hive-git


Description
-------

Currently it throws exception and follows non CBO path


Diffs
-----

  ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/CalciteSemanticException.java f6b4124

  ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/translator/RexNodeConverter.java
cb7bc26 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/translator/SqlFunctionConverter.java
cb86934 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/translator/TypeConverter.java a47010d

  ql/src/java/org/apache/hadoop/hive/ql/parse/TypeCheckProcFactory.java 96dab3b 
  ql/src/test/results/clientpositive/llap/vector_interval_1.q.out 0dd8fc5 
  ql/src/test/results/clientpositive/llap/vector_interval_arithmetic.q.out 8e3bc39 
  ql/src/test/results/clientpositive/vector_interval_1.q.out f18e56c 
  ql/src/test/results/clientpositive/vector_interval_arithmetic.q.out 5d1dca6 

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


Testing
-------

Existing regression suite


Thanks,

Ashutosh Chauhan


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