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

Switch to Plain View
Hive, mail # dev - Review Request 15113: HIVE-5356: Move arithmatic UDFs to generic UDF implementations


+
Xuefu Zhang 2013-10-31, 05:43
+
Jason Dere 2013-11-01, 01:45
+
Xuefu Zhang 2013-11-01, 01:56
+
Xuefu Zhang 2013-11-02, 03:43
+
Xuefu Zhang 2013-11-03, 19:26
+
Jason Dere 2013-11-05, 19:28
+
Jason Dere 2013-11-04, 22:56
+
Xuefu Zhang 2013-11-13, 15:04
+
Xuefu Zhang 2013-11-13, 15:04
+
Xuefu Zhang 2013-11-13, 15:33
+
Xuefu Zhang 2013-11-13, 15:34
+
Xuefu Zhang 2013-11-13, 15:35
+
Xuefu Zhang 2013-11-13, 15:35
+
Xuefu Zhang 2013-11-13, 15:36
+
Xuefu Zhang 2013-11-13, 15:36
+
Xuefu Zhang 2013-11-13, 15:33
+
Xuefu Zhang 2013-11-14, 20:44
+
Jason Dere 2013-11-14, 21:32
Copy link to this message
-
Re: Review Request 15113: HIVE-5356: Move arithmatic UDFs to generic UDF implementations
Xuefu Zhang 2013-11-14, 21:37


> On Nov. 14, 2013, 9:32 p.m., Jason Dere wrote:
> > ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFBaseNumeric.java, line 211
> > <https://reviews.apache.org/r/15113/diff/8/?file=384744#file384744line211>
> >
> >     This part is still incorrect for decimal types, see the following result types below, add/sub/mult is wrong when the precision/scale match.
> >    
> >                         expr: c1
> >                         type: decimal(10,2)
> >                         expr: c2
> >                         type: decimal(6,3)
> >                         expr: (c1 + c1)
> >                         type: decimal(10,2)
> >                         expr: (c1 + c2)
> >                         type: decimal(12,3)
> >                         expr: (c1 - c1)
> >                         type: decimal(10,2)
> >                         expr: (c1 - c2)
> >                         type: decimal(12,3)
> >                         expr: (c1 * c1)
> >                         type: decimal(10,2)
> >                         expr: (c1 * c2)
> >                         type: decimal(17,5)
> >                         expr: (c1 / c1)
> >                         type: decimal(23,13)
> >                         expr: (c1 / c2)
> >                         type: decimal(20,9)
> >

Got it. Thanks for pointing this out.
- Xuefu
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15113/#review28904
-----------------------------------------------------------
On Nov. 14, 2013, 8:44 p.m., Xuefu Zhang wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/15113/
> -----------------------------------------------------------
>
> (Updated Nov. 14, 2013, 8:44 p.m.)
>
>
> Review request for hive.
>
>
> Bugs: HIVe-5356
>     https://issues.apache.org/jira/browse/HIVe-5356
>
>
> Repository: hive-git
>
>
> Description
> -------
>
> Replace plus, minus, and so on 6 old UDFs with generic UDF implementations.
>
>
> Diffs
> -----
>
>   common/src/test/org/apache/hadoop/hive/common/type/TestHiveDecimal.java e3a1620
>   ql/src/java/org/apache/hadoop/hive/ql/exec/FunctionRegistry.java 5eb321c
>   ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/Vectorizer.java 7c1ab0d
>   ql/src/java/org/apache/hadoop/hive/ql/parse/TypeCheckProcFactory.java 7838547
>   ql/src/java/org/apache/hadoop/hive/ql/udf/UDFBaseNumericOp.java 1e74fce
>   ql/src/java/org/apache/hadoop/hive/ql/udf/UDFOPDivide.java b892b84
>   ql/src/java/org/apache/hadoop/hive/ql/udf/UDFOPMinus.java 0996231
>   ql/src/java/org/apache/hadoop/hive/ql/udf/UDFOPMod.java 0942ac3
>   ql/src/java/org/apache/hadoop/hive/ql/udf/UDFOPMultiply.java 07c1957
>   ql/src/java/org/apache/hadoop/hive/ql/udf/UDFOPPlus.java cfe7b2b
>   ql/src/java/org/apache/hadoop/hive/ql/udf/UDFPosMod.java 0da7eae
>   ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFBaseNumeric.java PRE-CREATION
>   ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFOPDivide.java PRE-CREATION
>   ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFOPMinus.java PRE-CREATION
>   ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFOPMod.java PRE-CREATION
>   ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFOPMultiply.java PRE-CREATION
>   ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFOPPlus.java PRE-CREATION
>   ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFPosMod.java PRE-CREATION
>   ql/src/test/org/apache/hadoop/hive/ql/exec/vector/TestVectorSelectOperator.java 4aeb4e6
>   ql/src/test/org/apache/hadoop/hive/ql/exec/vector/TestVectorizationContext.java 3f3e67f
>   ql/src/test/org/apache/hadoop/hive/ql/udf/TestUDFOPDivide.java 659d65f
>   ql/src/test/org/apache/hadoop/hive/ql/udf/TestUDFOPMod.java 4c2e722
>   ql/src/test/org/apache/hadoop/hive/ql/udf/TestUDFPosMod.java 2a301bc
+
Xuefu Zhang 2013-11-14, 22:11
+
Brock Noland 2013-11-15, 18:13
+
Xuefu Zhang 2013-11-17, 04:32