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/09/24 19:25:48 UTC

Review Request 25997: Collect table read entities at same time as view read entities

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

Review request for hive and Thejas Nair.


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


Repository: hive-git


Description
-------

Collect table read entities at same time as view read entities 


Diffs
-----

  ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java 2f36f04 
  ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java c4dacf9 

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


Testing
-------

Unit tests


Thanks,

Ashutosh Chauhan


Re: Review Request 25997: Collect table read entities at same time as view read entities

Posted by Thejas Nair <th...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25997/#review54523
-----------------------------------------------------------

Ship it!


Ship It!

- Thejas Nair


On Sept. 25, 2014, 6:41 a.m., Ashutosh Chauhan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25997/
> -----------------------------------------------------------
> 
> (Updated Sept. 25, 2014, 6:41 a.m.)
> 
> 
> Review request for hive and Thejas Nair.
> 
> 
> Bugs: HIVE-8245
>     https://issues.apache.org/jira/browse/HIVE-8245
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> Collect table read entities at same time as view read entities 
> 
> 
> Diffs
> -----
> 
>   ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java 2f36f04 
>   ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java c4dacf9 
>   ql/src/test/results/clientnegative/limit_partition_stats.q.out 5a5fe1f 
>   ql/src/test/results/clientpositive/alter_merge_stats_orc.q.out eae45b2 
>   ql/src/test/results/clientpositive/authorization_explain.q.out e5e605b 
>   ql/src/test/results/clientpositive/explain_dependency.q.out cb98d54 
>   ql/src/test/results/clientpositive/limit0.q.out d047374 
>   ql/src/test/results/clientpositive/limit_pushdown.q.out a5a0090 
>   ql/src/test/results/clientpositive/metadata_only_queries.q.out e273570 
>   ql/src/test/results/clientpositive/metadata_only_queries_with_filters.q.out 664e065 
>   ql/src/test/results/clientpositive/orc_analyze.q.out 07e46e9 
>   ql/src/test/results/clientpositive/orc_merge5.q.out 2ac3342 
>   ql/src/test/results/clientpositive/orc_merge6.q.out 05deb57 
>   ql/src/test/results/clientpositive/orc_merge7.q.out d342736 
>   ql/src/test/results/clientpositive/orc_merge_incompat1.q.out e6ef838 
>   ql/src/test/results/clientpositive/orc_merge_incompat2.q.out e28d8b3 
>   ql/src/test/results/clientpositive/ql_rewrite_gbtoidx.q.out 0e7c4af 
>   ql/src/test/results/clientpositive/query_properties.q.out 47f8d8c 
>   ql/src/test/results/clientpositive/stats_only_null.q.out c4728c9 
>   ql/src/test/results/clientpositive/tez/alter_merge_stats_orc.q.out eae45b2 
>   ql/src/test/results/clientpositive/tez/limit_pushdown.q.out 23df5ec 
>   ql/src/test/results/clientpositive/tez/metadata_only_queries.q.out 7942ce7 
>   ql/src/test/results/clientpositive/tez/orc_analyze.q.out 07e46e9 
>   ql/src/test/results/clientpositive/tez/orc_merge5.q.out b40a37d 
>   ql/src/test/results/clientpositive/tez/orc_merge6.q.out 0441fa4 
>   ql/src/test/results/clientpositive/tez/orc_merge7.q.out c6809a1 
>   ql/src/test/results/clientpositive/tez/orc_merge_incompat1.q.out 90f7f24 
>   ql/src/test/results/clientpositive/tez/orc_merge_incompat2.q.out 30c6ab8 
>   ql/src/test/results/clientpositive/vectorization_limit.q.out d67d559 
> 
> Diff: https://reviews.apache.org/r/25997/diff/
> 
> 
> Testing
> -------
> 
> Unit tests
> 
> 
> Thanks,
> 
> Ashutosh Chauhan
> 
>


Re: Review Request 25997: Collect table read entities at same time as view read entities

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

(Updated Sept. 25, 2014, 6:41 a.m.)


Review request for hive and Thejas Nair.


Changes
-------

Updated patch with golden files update.


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


Repository: hive-git


Description
-------

Collect table read entities at same time as view read entities 


Diffs (updated)
-----

  ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java 2f36f04 
  ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java c4dacf9 
  ql/src/test/results/clientnegative/limit_partition_stats.q.out 5a5fe1f 
  ql/src/test/results/clientpositive/alter_merge_stats_orc.q.out eae45b2 
  ql/src/test/results/clientpositive/authorization_explain.q.out e5e605b 
  ql/src/test/results/clientpositive/explain_dependency.q.out cb98d54 
  ql/src/test/results/clientpositive/limit0.q.out d047374 
  ql/src/test/results/clientpositive/limit_pushdown.q.out a5a0090 
  ql/src/test/results/clientpositive/metadata_only_queries.q.out e273570 
  ql/src/test/results/clientpositive/metadata_only_queries_with_filters.q.out 664e065 
  ql/src/test/results/clientpositive/orc_analyze.q.out 07e46e9 
  ql/src/test/results/clientpositive/orc_merge5.q.out 2ac3342 
  ql/src/test/results/clientpositive/orc_merge6.q.out 05deb57 
  ql/src/test/results/clientpositive/orc_merge7.q.out d342736 
  ql/src/test/results/clientpositive/orc_merge_incompat1.q.out e6ef838 
  ql/src/test/results/clientpositive/orc_merge_incompat2.q.out e28d8b3 
  ql/src/test/results/clientpositive/ql_rewrite_gbtoidx.q.out 0e7c4af 
  ql/src/test/results/clientpositive/query_properties.q.out 47f8d8c 
  ql/src/test/results/clientpositive/stats_only_null.q.out c4728c9 
  ql/src/test/results/clientpositive/tez/alter_merge_stats_orc.q.out eae45b2 
  ql/src/test/results/clientpositive/tez/limit_pushdown.q.out 23df5ec 
  ql/src/test/results/clientpositive/tez/metadata_only_queries.q.out 7942ce7 
  ql/src/test/results/clientpositive/tez/orc_analyze.q.out 07e46e9 
  ql/src/test/results/clientpositive/tez/orc_merge5.q.out b40a37d 
  ql/src/test/results/clientpositive/tez/orc_merge6.q.out 0441fa4 
  ql/src/test/results/clientpositive/tez/orc_merge7.q.out c6809a1 
  ql/src/test/results/clientpositive/tez/orc_merge_incompat1.q.out 90f7f24 
  ql/src/test/results/clientpositive/tez/orc_merge_incompat2.q.out 30c6ab8 
  ql/src/test/results/clientpositive/vectorization_limit.q.out d67d559 

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


Testing
-------

Unit tests


Thanks,

Ashutosh Chauhan


Re: Review Request 25997: Collect table read entities at same time as view read entities

Posted by Thejas Nair <th...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25997/#review54472
-----------------------------------------------------------

Ship it!


Ship It!

- Thejas Nair


On Sept. 24, 2014, 5:25 p.m., Ashutosh Chauhan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25997/
> -----------------------------------------------------------
> 
> (Updated Sept. 24, 2014, 5:25 p.m.)
> 
> 
> Review request for hive and Thejas Nair.
> 
> 
> Bugs: HIVE-8245
>     https://issues.apache.org/jira/browse/HIVE-8245
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> Collect table read entities at same time as view read entities 
> 
> 
> Diffs
> -----
> 
>   ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java 2f36f04 
>   ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java c4dacf9 
> 
> Diff: https://reviews.apache.org/r/25997/diff/
> 
> 
> Testing
> -------
> 
> Unit tests
> 
> 
> Thanks,
> 
> Ashutosh Chauhan
> 
>