svn commit: r1550802 - /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFLog10.java

[hashutosh]

Author: hashutosh
Date: Fri Dec 13 18:45:39 2013
New Revision: 1550802

URL: http://svn.apache.org/r1550802
Log:
HIVE-6027 : non-vectorized log10 has rounding issue (Sergey Shelukhin via 
Ashutosh Chauhan)

Modified:
    hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFLog10.java

Modified: hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFLog10.java
URL: 
http://svn.apache.org/viewvc/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFLog10.java?rev=1550802&r1=1550801&r2=1550802&view=diff
==============================================================================
--- hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFLog10.java 
(original)
+++ hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFLog10.java Fri Dec 
13 18:45:39 2013
@@ -35,8 +35,6 @@ import org.apache.hadoop.hive.serde2.io.
     + "  > SELECT _FUNC_(10) FROM src LIMIT 1;\n" + "  1")
 @VectorizedExpressions({FuncLog10LongToDouble.class, 
FuncLog10DoubleToDouble.class})
 public class UDFLog10 extends UDF {
-  private static double log10 = Math.log(10.0);
-
   private final DoubleWritable result = new DoubleWritable();
 
   public UDFLog10() {
@@ -49,9 +47,8 @@ public class UDFLog10 extends UDF {
     if (a == null || a.get() <= 0.0) {
       return null;
     } else {
-      result.set(Math.log(a.get()) / log10);
+      result.set(Math.log10(a.get()));
       return result;
     }
   }
-
 }