sagiruthvik commented on code in PR #3122:
URL: https://github.com/apache/fory/pull/3122#discussion_r2699275594


##########
java/fory-core/src/main/java/org/apache/fory/Fory.java:
##########
@@ -105,6 +106,7 @@ public final class Fory implements BaseFory {
   private static final boolean isLittleEndian = ByteOrder.nativeOrder() == 
ByteOrder.LITTLE_ENDIAN;
   private static final byte BITMAP = isLittleEndian ? isLittleEndianFlag : 0;
   private static final short MAGIC_NUMBER = 0x62D4;
+  private static final AtomicInteger runtimeHashCounter = new AtomicInteger(0);

Review Comment:
   I've updated the implementation to compute the hash based on registered 
serializers instead of using a counter. Now configHash is calculated from 
config.hashCode() plus the hash of all registered serializer class names. This 
way, pooled Fury instances with the same serializers will share the same hash, 
while instances with different serializers get different hashes.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


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

Reply via email to