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 2015/06/27 23:25:05 UTC

Review Request 35969: HIVE-11128

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

Review request for hive and Prasanth_J.


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


Repository: hive-git


Description
-------

Stats Annotation misses extracting stats for cols in some cases.


Diffs
-----

  ql/src/java/org/apache/hadoop/hive/ql/stats/StatsUtils.java 0bd7f0a 
  ql/src/test/results/clientnegative/udf_assert_true.q.out 819d723 
  ql/src/test/results/clientnegative/udf_assert_true2.q.out 9760d0d 
  ql/src/test/results/clientpositive/annotate_stats_groupby.q.out b9be798 
  ql/src/test/results/clientpositive/annotate_stats_groupby2.q.out c4293c0 
  ql/src/test/results/clientpositive/annotate_stats_join_pkfk.q.out dd70708 
  ql/src/test/results/clientpositive/annotate_stats_select.q.out c7575d1 
  ql/src/test/results/clientpositive/cross_join.q.out 377fde9 
  ql/src/test/results/clientpositive/input_part10.q.out f4b959d 
  ql/src/test/results/clientpositive/insert_into5.q.out b01f761 
  ql/src/test/results/clientpositive/lateral_view.q.out ce20041 
  ql/src/test/results/clientpositive/lateral_view_explode2.q.out 4e3e021 
  ql/src/test/results/clientpositive/lateral_view_noalias.q.out ebd6bc5 
  ql/src/test/results/clientpositive/load_dyn_part14.q.out 3dcc3c9 
  ql/src/test/results/clientpositive/spark/cross_join.q.out 2fb6b21 
  ql/src/test/results/clientpositive/spark/lateral_view_explode2.q.out 1ac37a9 
  ql/src/test/results/clientpositive/spark/load_dyn_part14.q.out 4e030a3 
  ql/src/test/results/clientpositive/tez/cross_join.q.out 30564d7 
  ql/src/test/results/clientpositive/tez/explainuser_1.q.out bd5b72e 
  ql/src/test/results/clientpositive/tez/explainuser_2.q.out 95924aa 
  ql/src/test/results/clientpositive/tez/insert1.q.out e3088e0 
  ql/src/test/results/clientpositive/tez/tez_union_dynamic_partition.q.out 691ab17 
  ql/src/test/results/clientpositive/tez/tez_union_multiinsert.q.out 132f859 
  ql/src/test/results/clientpositive/tez/vector_decimal_2.q.out f17076c 
  ql/src/test/results/clientpositive/udtf_stack.q.out c8101e6 
  ql/src/test/results/clientpositive/vector_decimal_2.q.out a13793d 

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


Testing
-------

Existing tests.


Thanks,

Ashutosh Chauhan


Re: Review Request 35969: HIVE-11128

Posted by j....@gmail.com.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35969/#review89637
-----------------------------------------------------------

Ship it!


Ship It!

- Prasanth_J


On June 27, 2015, 9:25 p.m., Ashutosh Chauhan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/35969/
> -----------------------------------------------------------
> 
> (Updated June 27, 2015, 9:25 p.m.)
> 
> 
> Review request for hive and Prasanth_J.
> 
> 
> Bugs: HIVE-11128
>     https://issues.apache.org/jira/browse/HIVE-11128
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> Stats Annotation misses extracting stats for cols in some cases.
> 
> 
> Diffs
> -----
> 
>   ql/src/java/org/apache/hadoop/hive/ql/stats/StatsUtils.java 0bd7f0a 
>   ql/src/test/results/clientnegative/udf_assert_true.q.out 819d723 
>   ql/src/test/results/clientnegative/udf_assert_true2.q.out 9760d0d 
>   ql/src/test/results/clientpositive/annotate_stats_groupby.q.out b9be798 
>   ql/src/test/results/clientpositive/annotate_stats_groupby2.q.out c4293c0 
>   ql/src/test/results/clientpositive/annotate_stats_join_pkfk.q.out dd70708 
>   ql/src/test/results/clientpositive/annotate_stats_select.q.out c7575d1 
>   ql/src/test/results/clientpositive/cross_join.q.out 377fde9 
>   ql/src/test/results/clientpositive/input_part10.q.out f4b959d 
>   ql/src/test/results/clientpositive/insert_into5.q.out b01f761 
>   ql/src/test/results/clientpositive/lateral_view.q.out ce20041 
>   ql/src/test/results/clientpositive/lateral_view_explode2.q.out 4e3e021 
>   ql/src/test/results/clientpositive/lateral_view_noalias.q.out ebd6bc5 
>   ql/src/test/results/clientpositive/load_dyn_part14.q.out 3dcc3c9 
>   ql/src/test/results/clientpositive/spark/cross_join.q.out 2fb6b21 
>   ql/src/test/results/clientpositive/spark/lateral_view_explode2.q.out 1ac37a9 
>   ql/src/test/results/clientpositive/spark/load_dyn_part14.q.out 4e030a3 
>   ql/src/test/results/clientpositive/tez/cross_join.q.out 30564d7 
>   ql/src/test/results/clientpositive/tez/explainuser_1.q.out bd5b72e 
>   ql/src/test/results/clientpositive/tez/explainuser_2.q.out 95924aa 
>   ql/src/test/results/clientpositive/tez/insert1.q.out e3088e0 
>   ql/src/test/results/clientpositive/tez/tez_union_dynamic_partition.q.out 691ab17 
>   ql/src/test/results/clientpositive/tez/tez_union_multiinsert.q.out 132f859 
>   ql/src/test/results/clientpositive/tez/vector_decimal_2.q.out f17076c 
>   ql/src/test/results/clientpositive/udtf_stack.q.out c8101e6 
>   ql/src/test/results/clientpositive/vector_decimal_2.q.out a13793d 
> 
> Diff: https://reviews.apache.org/r/35969/diff/
> 
> 
> Testing
> -------
> 
> Existing tests.
> 
> 
> Thanks,
> 
> Ashutosh Chauhan
> 
>