You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Ashutosh Chauhan <ha...@apache.org> on 2014/08/21 08:17:05 UTC

Review Request 24920: CBO path doesn't handle null expr in select list correctly.

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

Review request for hive and John Pullokkaran.


Repository: hive


Description
-------

CBO path doesn't handle null expr in select list correctly.


Diffs
-----

  branches/cbo/ql/src/java/org/apache/hadoop/hive/ql/optimizer/optiq/translator/ASTBuilder.java 1619293 
  branches/cbo/ql/src/java/org/apache/hadoop/hive/ql/optimizer/optiq/translator/ASTConverter.java 1619294 
  branches/cbo/ql/src/java/org/apache/hadoop/hive/ql/optimizer/optiq/translator/RexNodeConverter.java 1619293 
  branches/cbo/ql/src/java/org/apache/hadoop/hive/ql/optimizer/optiq/translator/TypeConverter.java 1619293 
  branches/cbo/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 1619294 
  branches/cbo/ql/src/test/queries/clientpositive/cbo_correctness.q 1619294 
  branches/cbo/ql/src/test/results/clientpositive/cbo_correctness.q.out 1619294 

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


Testing
-------

added new test.


Thanks,

Ashutosh Chauhan


Re: Review Request 24920: CBO path doesn't handle null expr in select list correctly.

Posted by Ashutosh Chauhan <ha...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/24920/
-----------------------------------------------------------

(Updated Aug. 21, 2014, 6:28 a.m.)


Review request for hive and John Pullokkaran.


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


Repository: hive


Description (updated)
-------

CBO path doesn't handle null expr in select list correctly


Diffs
-----

  branches/cbo/ql/src/java/org/apache/hadoop/hive/ql/optimizer/optiq/translator/ASTBuilder.java 1619293 
  branches/cbo/ql/src/java/org/apache/hadoop/hive/ql/optimizer/optiq/translator/ASTConverter.java 1619294 
  branches/cbo/ql/src/java/org/apache/hadoop/hive/ql/optimizer/optiq/translator/RexNodeConverter.java 1619293 
  branches/cbo/ql/src/java/org/apache/hadoop/hive/ql/optimizer/optiq/translator/TypeConverter.java 1619293 
  branches/cbo/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 1619294 
  branches/cbo/ql/src/test/queries/clientpositive/cbo_correctness.q 1619294 
  branches/cbo/ql/src/test/results/clientpositive/cbo_correctness.q.out 1619294 

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


Testing
-------

added new test.


Thanks,

Ashutosh Chauhan