You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Navis Ryu <na...@nexr.com> on 2014/12/29 05:50:57 UTC
Re: Review Request 23351: Support direct fetch for lateral views,
sub queries, etc.
> On Nov. 3, 2014, 10:28 p.m., John Pullokkaran wrote:
> > ql/src/java/org/apache/hadoop/hive/ql/optimizer/SimpleFetchOptimizer.java, line 162
> > <https://reviews.apache.org/r/23351/diff/1/?file=626500#file626500line162>
> >
> > Can't we use ParseContext.topToTable to get to "Table" given a TS object instead of walking the QB tree?
Good point! Sorry for missing this comment. I'll update the patch shortly after.
- Navis
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/23351/#review59662
-----------------------------------------------------------
On July 9, 2014, 6:55 a.m., Navis Ryu wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/23351/
> -----------------------------------------------------------
>
> (Updated July 9, 2014, 6:55 a.m.)
>
>
> Review request for hive.
>
>
> Bugs: HIVE-5718
> https://issues.apache.org/jira/browse/HIVE-5718
>
>
> Repository: hive-git
>
>
> Description
> -------
>
> Extend HIVE-2925 with LV and SubQ.
>
>
> Diffs
> -----
>
> ql/src/java/org/apache/hadoop/hive/ql/exec/OperatorFactory.java 5d41fa1
> ql/src/java/org/apache/hadoop/hive/ql/optimizer/SimpleFetchOptimizer.java 7413d2b
> ql/src/java/org/apache/hadoop/hive/ql/parse/QB.java 908db1e
> ql/src/java/org/apache/hadoop/hive/ql/parse/QBParseInfo.java 911ac8a
> ql/src/java/org/apache/hadoop/hive/ql/plan/FetchWork.java 32d84ea
> ql/src/test/queries/clientpositive/nonmr_fetch.q 2a92d17
> ql/src/test/queries/clientpositive/nonmr_fetch_threshold.q e6343e2
> ql/src/test/results/clientpositive/explain_logical.q.out bb26e8c
> ql/src/test/results/clientpositive/lateral_view_noalias.q.out d51b2de
> ql/src/test/results/clientpositive/nonmr_fetch.q.out 5a13e84
> ql/src/test/results/clientpositive/nonmr_fetch_threshold.q.out 39cdfa6
> ql/src/test/results/clientpositive/select_dummy_source.q.out 2742d56
> ql/src/test/results/clientpositive/subquery_alias.q.out 37bc3a4
> ql/src/test/results/clientpositive/udf_explode.q.out 4eeedeb
> ql/src/test/results/clientpositive/udf_inline.q.out e065bed
> ql/src/test/results/clientpositive/udf_reflect2.q.out 6b19277
> ql/src/test/results/clientpositive/udf_to_unix_timestamp.q.out 447ef87
> ql/src/test/results/clientpositive/udtf_explode.q.out ae95907
>
> Diff: https://reviews.apache.org/r/23351/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Navis Ryu
>
>