This is an automated email from the ASF dual-hosted git repository.

leerho pushed a commit to branch hash_bug
in repository https://gitbox.apache.org/repos/asf/datasketches-memory.git


The following commit(s) were added to refs/heads/hash_bug by this push:
     new 38df8a7b Only change required to fix the Hash Bug.
38df8a7b is described below

commit 38df8a7b734f5713b755e1999975c6dde15318b2
Author: Lee Rhodes <[email protected]>
AuthorDate: Tue Feb 11 16:50:07 2025 -0800

    Only change required to fix the Hash Bug.
    
    Does not include adding tests for it.
---
 .../java/org/apache/datasketches/memory/internal/MurmurHash3v4.java   | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/src/main/java/org/apache/datasketches/memory/internal/MurmurHash3v4.java 
b/src/main/java/org/apache/datasketches/memory/internal/MurmurHash3v4.java
index 4948464a..2628bc07 100644
--- a/src/main/java/org/apache/datasketches/memory/internal/MurmurHash3v4.java
+++ b/src/main/java/org/apache/datasketches/memory/internal/MurmurHash3v4.java
@@ -243,11 +243,11 @@ public final class MurmurHash3v4 {
         }
 
         case 13: {
-          k2 ^= (seg.get(ValueLayout.JAVA_BYTE, cumOff + 12) & 0xFFFFL) << 32;
+          k2 ^= (seg.get(ValueLayout.JAVA_BYTE, cumOff + 12) & 0xFFL) << 32;
         }
         //$FALL-THROUGH$
         case 12: {
-          k2 ^= (seg.get(ValueLayout.JAVA_INT_UNALIGNED, cumOff + 8) & 
0xFFFFFFFFL);
+          k2 ^= seg.get(ValueLayout.JAVA_INT_UNALIGNED, cumOff + 8) & 
0xFFFFFFFFL;
           k1 = seg.get(ValueLayout.JAVA_LONG_UNALIGNED, cumOff);
           break;
         }


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to