Fix comment for Timestamp in HiveBucketing

prestodb · Jun 10, 2024 · 3d06a01 · 3d06a01
1 parent fb2a314
commit 3d06a01
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/presto-hive/src/main/java/com/facebook/presto/hive/HiveBucketing.java b/presto-hive/src/main/java/com/facebook/presto/hive/HiveBucketing.java
@@ -174,9 +174,9 @@ private static int hash(TypeInfo type, Block block, int position)
                         return toIntExact(days);
                     case TIMESTAMP:
                         long millisSinceEpoch = prestoType.getLong(block, position);
-                        // seconds << 30 + nanoseconds
-                        long secondsAndNanos = (Math.floorDiv(millisSinceEpoch, 1000L) << 30) + Math.floorMod(millisSinceEpoch, 1000L);
-                        return (int) ((secondsAndNanos >>> 32) ^ secondsAndNanos);
+                        // seconds << 30 + milliseconds
+                        long secondsAndMillis = (Math.floorDiv(millisSinceEpoch, 1000L) << 30) + Math.floorMod(millisSinceEpoch, 1000L);
+                        return (int) ((secondsAndMillis >>> 32) ^ secondsAndMillis);
                     default:
                         throw new UnsupportedOperationException("Computation of Hive bucket hashCode is not supported for Hive primitive category: " + primitiveCategory.toString() + ".");
                 }
@@ -233,9 +233,9 @@ private static int hash(TypeInfo type, Object value)
                         return toIntExact(days);
                     case TIMESTAMP:
                         long millisSinceEpoch = (long) value;
-                        // seconds << 30 + nanoseconds
-                        long secondsAndNanos = (Math.floorDiv(millisSinceEpoch, 1000L) << 30) + Math.floorMod(millisSinceEpoch, 1000L);
-                        return (int) ((secondsAndNanos >>> 32) ^ secondsAndNanos);
+                        // seconds << 30 + milliseconds
+                        long secondsAndMillis = (Math.floorDiv(millisSinceEpoch, 1000L) << 30) + Math.floorMod(millisSinceEpoch, 1000L);
+                        return (int) ((secondsAndMillis >>> 32) ^ secondsAndMillis);
                     default:
                         throw new UnsupportedOperationException("Computation of Hive bucket hashCode is not supported for Hive primitive category: " + primitiveCategory.toString() + ".");
                 }