hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Harish Butani" <rhbut...@gmail.com>
Subject Re: Review Request 20737: HIVE-6031: explain subquery rewrite for where clause predicates
Date Tue, 29 Apr 2014 00:43:36 GMT

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

(Updated April 29, 2014, 12:43 a.m.)


Review request for hive and Ashutosh Chauhan.


Changes
-------

address review comments


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


Repository: hive-git


Description
-------

explain subquery rewrite for where clause predicates


Diffs (updated)
-----

  ql/src/java/org/apache/hadoop/hive/ql/exec/ExplainSQRewriteTask.java PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/exec/TaskFactory.java 679c6ec 
  ql/src/java/org/apache/hadoop/hive/ql/parse/ExplainSQRewriteSemanticAnalyzer.java PRE-CREATION

  ql/src/java/org/apache/hadoop/hive/ql/parse/HiveLexer.g 3e673ca 
  ql/src/java/org/apache/hadoop/hive/ql/parse/HiveParser.g 13bbf0a 
  ql/src/java/org/apache/hadoop/hive/ql/parse/IdentifiersParser.g 864e692 
  ql/src/java/org/apache/hadoop/hive/ql/parse/QB.java a8b436e 
  ql/src/java/org/apache/hadoop/hive/ql/parse/QBSubQuery.java b7c9e65 
  ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 0e0395e 
  ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzerFactory.java f6b70d8 
  ql/src/java/org/apache/hadoop/hive/ql/parse/SubQueryDiagnostic.java PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/parse/SubQueryUtils.java 07d32ed 
  ql/src/java/org/apache/hadoop/hive/ql/plan/ExplainSQRewriteWork.java PRE-CREATION 
  ql/src/test/queries/clientpositive/subquery_exists_explain_rewrite.q PRE-CREATION 
  ql/src/test/queries/clientpositive/subquery_in_explain_rewrite.q PRE-CREATION 
  ql/src/test/results/clientpositive/subquery_exists_explain_rewrite.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/subquery_in_explain_rewrite.q.out PRE-CREATION 

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


Testing
-------

new tests added


Thanks,

Harish Butani


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