Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Plain View
Hive, mail # dev - Review Request 15103: HIVe-4523: round() function with specified decimal places not consistent with mysql


+
Xuefu Zhang 2013-10-30, 20:17
Copy link to this message
-
Re: Review Request 15103: HIVe-4523: round() function with specified decimal places not consistent with mysql
Brock Noland 2013-11-04, 21:33

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

Ship it!
Ship It!

- Brock Noland
On Oct. 30, 2013, 8:17 p.m., Xuefu Zhang wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/15103/
> -----------------------------------------------------------
>
> (Updated Oct. 30, 2013, 8:17 p.m.)
>
>
> Review request for hive.
>
>
> Bugs: HIVE-4523
>     https://issues.apache.org/jira/browse/HIVE-4523
>
>
> Repository: hive-git
>
>
> Description
> -------
>
> 1. Migrate old, non-generic round udf to generic udf.
> 2. Addressed issues raised in the description.
>
>
> Diffs
> -----
>
>   ql/src/java/org/apache/hadoop/hive/ql/exec/FunctionRegistry.java 898b6a5
>   ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/RoundWithNumDigitsDoubleToDouble.java ca85688
>   ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/Vectorizer.java 120095a
>   ql/src/java/org/apache/hadoop/hive/ql/udf/UDFRound.java 6d5664b
>   ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFRound.java PRE-CREATION
>   ql/src/java/org/apache/hadoop/hive/ql/udf/generic/RoundUtils.java PRE-CREATION
>   ql/src/test/org/apache/hadoop/hive/ql/exec/vector/TestVectorizationContext.java 90ab983
>   ql/src/test/org/apache/hadoop/hive/ql/udf/TestUDFRound.java 992853d
>   ql/src/test/org/apache/hadoop/hive/ql/udf/generic/TestGenericUDFRound.java PRE-CREATION
>   ql/src/test/results/clientpositive/decimal_udf.q.out 652d1a8
>   ql/src/test/results/compiler/plan/udf4.q.xml b962f7e
>
> Diff: https://reviews.apache.org/r/15103/diff/
>
>
> Testing
> -------
>
> Added test cases. Some old q test outputs are regenerated.
>
>
> Thanks,
>
> Xuefu Zhang
>
>