----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16281/ -----------------------------------------------------------
Review request for hive. Bugs: HIVE-4293 https://issues.apache.org/jira/browse/HIVE-4293 Repository: hive-git Description ------- For example, {noformat} explain SELECT value from ( select explode(array(key, value)) as (value) from ( select * FROM src WHERE key > 200 ) A ) B WHERE value > 300 ; {noformat} Makes plan like this, removing last predicates {noformat} TableScan alias: src Filter Operator predicate: expr: (key > 200.0) type: boolean Select Operator expressions: expr: array(key,value) type: array<string> outputColumnNames: _col0 UDTF Operator function name: explode Select Operator expressions: expr: col type: string outputColumnNames: _col0 File Output Operator compressed: false GlobalTableId: 0 table: input format: org.apache.hadoop.mapred.TextInputFormat output format: org.apache.hadoop.hive.ql.io.HiveIgnoreKeyTextOutputFormat {noformat} Diffs ----- ql/src/java/org/apache/hadoop/hive/ql/exec/LateralViewJoinOperator.java 2fbb81b ql/src/java/org/apache/hadoop/hive/ql/exec/ReduceSinkOperator.java c378dc7 ql/src/java/org/apache/hadoop/hive/ql/optimizer/ColumnPrunerProcFactory.java 0798470 ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java d0a0ec7 ql/src/java/org/apache/hadoop/hive/ql/plan/LateralViewJoinDesc.java ebfcfc8 ql/src/java/org/apache/hadoop/hive/ql/ppd/ExprWalkerInfo.java 6a3dd99 ql/src/java/org/apache/hadoop/hive/ql/ppd/OpProcFactory.java 40298e1 ql/src/java/org/apache/hadoop/hive/ql/ppd/PredicatePushDown.java cd5ae51 ql/src/test/queries/clientpositive/lateral_view_ppd.q 7be86a6 ql/src/test/queries/clientpositive/ppd_udtf.q PRE-CREATION ql/src/test/results/clientpositive/cluster.q.out 8d14a1d ql/src/test/results/clientpositive/ctas_colname.q.out a15b698 ql/src/test/results/clientpositive/lateral_view_ppd.q.out f54c809 ql/src/test/results/clientpositive/ppd2.q.out f6af8f8 ql/src/test/results/clientpositive/ppd_gby.q.out 5908450 ql/src/test/results/clientpositive/ppd_gby2.q.out bdd7e89 ql/src/test/results/clientpositive/ppd_udtf.q.out PRE-CREATION ql/src/test/results/clientpositive/udtf_json_tuple.q.out 1a480b6 ql/src/test/results/clientpositive/udtf_parse_url_tuple.q.out a38b31b ql/src/test/results/compiler/plan/join1.q.xml 9e4b609 ql/src/test/results/compiler/plan/join2.q.xml efcb865 ql/src/test/results/compiler/plan/join3.q.xml 9bbe64f ql/src/test/results/compiler/plan/join4.q.xml f8a8f10 ql/src/test/results/compiler/plan/join5.q.xml d43ce00 ql/src/test/results/compiler/plan/join6.q.xml a354e4b ql/src/test/results/compiler/plan/join7.q.xml ac41995 ql/src/test/results/compiler/plan/join8.q.xml 569cf6e Diff: https://reviews.apache.org/r/16281/diff/ Testing ------- Thanks, Navis Ryu