You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Mohit Sabharwal <mo...@cloudera.com> on 2016/08/05 20:24:25 UTC
Re: Review Request 50405: HIVE-14251: Union All of different types
resolves to incorrect data
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50405/#review144977
-----------------------------------------------------------
Ship it!
Ship It!
- Mohit Sabharwal
On July 26, 2016, 3:13 p.m., Aihua Xu wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/50405/
> -----------------------------------------------------------
>
> (Updated July 26, 2016, 3:13 p.m.)
>
>
> Review request for hive.
>
>
> Repository: hive-git
>
>
> Description
> -------
>
> HIVE-14251: Union All of different types resolves to incorrect data
>
>
> Diffs
> -----
>
> ql/src/java/org/apache/hadoop/hive/ql/exec/FunctionRegistry.java 69a18cd
> ql/src/test/org/apache/hadoop/hive/ql/exec/TestFunctionRegistry.java 59ecd1e
> ql/src/test/queries/clientpositive/alter_partition_change_col.q 360f4d2
> ql/src/test/queries/clientpositive/alter_table_cascade.q acca4e8
> ql/src/test/queries/clientpositive/groupby_sort_1_23.q 67fdd23
> ql/src/test/queries/clientpositive/groupby_sort_skew_1_23.q 39b9420
> ql/src/test/queries/clientpositive/union32.q f47f0af
> ql/src/test/queries/clientpositive/union33.q 017442e
> ql/src/test/queries/clientpositive/union36.q c38e7b1
> ql/src/test/queries/clientpositive/unionDistinct_1.q fd7f075
> ql/src/test/queries/clientpositive/union_date_trim.q 6842e56
> ql/src/test/queries/clientpositive/union_null.q a17325c
> ql/src/test/queries/clientpositive/union_remove_12.q f6436f5
> ql/src/test/queries/clientpositive/union_remove_13.q b02451b
> ql/src/test/queries/clientpositive/union_remove_14.q bec6226
> ql/src/test/queries/clientpositive/union_type_chk.q 3b7b478
> ql/src/test/queries/clientpositive/unionall_join_nullconstant.q 4f0ffa6
> ql/src/test/results/clientpositive/alter_partition_change_col.q.out 23febee
> ql/src/test/results/clientpositive/alter_table_cascade.q.out 1d8204c
> ql/src/test/results/clientpositive/groupby_sort_1_23.q.out 81fe0d9
> ql/src/test/results/clientpositive/groupby_sort_skew_1_23.q.out 5cf0ea2
> ql/src/test/results/clientpositive/spark/groupby_sort_1_23.q.out 408c1b9
> ql/src/test/results/clientpositive/spark/groupby_sort_skew_1_23.q.out 6325889
> ql/src/test/results/clientpositive/spark/union32.q.out 1ec7e64
> ql/src/test/results/clientpositive/spark/union33.q.out a61a8df
> ql/src/test/results/clientpositive/spark/union_date_trim.q.out 324e8b7
> ql/src/test/results/clientpositive/spark/union_null.q.out 32cdf65
> ql/src/test/results/clientpositive/spark/union_remove_12.q.out 94b4211
> ql/src/test/results/clientpositive/spark/union_remove_13.q.out 42aea66
> ql/src/test/results/clientpositive/spark/union_remove_14.q.out cf6d36f
> ql/src/test/results/clientpositive/tez/unionDistinct_1.q.out ee33086
> ql/src/test/results/clientpositive/tez/union_type_chk.q.out 12f060b
> ql/src/test/results/clientpositive/union32.q.out a3fefa8
> ql/src/test/results/clientpositive/union33.q.out a91e74c
> ql/src/test/results/clientpositive/union36.q.out 12f060b
> ql/src/test/results/clientpositive/unionDistinct_1.q.out 0330133
> ql/src/test/results/clientpositive/union_date_trim.q.out 324e8b7
> ql/src/test/results/clientpositive/union_null.q.out 32cdf65
> ql/src/test/results/clientpositive/union_remove_12.q.out 5f73c9a
> ql/src/test/results/clientpositive/union_remove_13.q.out c7063cd
> ql/src/test/results/clientpositive/union_remove_14.q.out 52dc7c5
> ql/src/test/results/clientpositive/union_type_chk.q.out 12f060b
> ql/src/test/results/clientpositive/unionall_join_nullconstant.q.out fca26b4
> serde/src/java/org/apache/hadoop/hive/serde2/typeinfo/TypeInfoUtils.java 8f7b799
>
> Diff: https://reviews.apache.org/r/50405/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Aihua Xu
>
>