Hash object values, rather than ordinals

stevenewald · Jul 17, 2023 · 51cc570 · 51cc570
1 parent adf1eb2
commit 51cc570
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/hollow/src/main/java/com/netflix/hollow/tools/history/keyindex/HollowOrdinalMapper.java b/hollow/src/main/java/com/netflix/hollow/tools/history/keyindex/HollowOrdinalMapper.java
@@ -167,7 +167,8 @@ public Object getFieldObject(int keyOrdinal, int fieldIndex) {
     private int hashKeyRecord(HollowObjectTypeReadState typeState, int ordinal) {
         int hashCode = 0;
         for (int i = 0; i < primaryKey.numFields(); i++) {
-            int fieldHashCode = HollowReadFieldUtils.fieldHashCode(typeState, ordinal, i);
+            Object fieldObjectToHash = readValueInState(typeState, ordinal, i);
+            int fieldHashCode = HollowReadFieldUtils.hashObject(fieldObjectToHash);
             hashCode = (hashCode * 31) ^ fieldHashCode;
         }
         return HashCodes.hashInt(hashCode);