You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by ha...@apache.org on 2013/08/05 23:07:39 UTC

svn commit: r1510733 - /hive/branches/vectorization/ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/VectorUDFUnixTimeStampLong.java

Author: hashutosh
Date: Mon Aug  5 21:07:38 2013
New Revision: 1510733

URL: http://svn.apache.org/r1510733
Log:
HIVE-4971 : Unit test failure in TestVectorTimestampExpressions (Jitendra Nath Pandey via Ashutosh Chauhan)

Modified:
    hive/branches/vectorization/ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/VectorUDFUnixTimeStampLong.java

Modified: hive/branches/vectorization/ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/VectorUDFUnixTimeStampLong.java
URL: http://svn.apache.org/viewvc/hive/branches/vectorization/ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/VectorUDFUnixTimeStampLong.java?rev=1510733&r1=1510732&r2=1510733&view=diff
==============================================================================
--- hive/branches/vectorization/ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/VectorUDFUnixTimeStampLong.java (original)
+++ hive/branches/vectorization/ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/VectorUDFUnixTimeStampLong.java Mon Aug  5 21:07:38 2013
@@ -26,11 +26,11 @@ public final class VectorUDFUnixTimeStam
 
   @Override
   protected long getField(long time) {
-    long ms = time / (1000*1000);
-    long remainder = time % (1000*1000);
+    long ms = (time / (1000*1000*1000)) * 1000;
+    long remainder = time % (1000*1000*1000);
     /* negative timestamps need to be adjusted */
     if(remainder < 0) {
-      ms -= 1;
+      ms -= 1000;
     }
     return ms/1000;
   }