[GitHub] [jena] Claudenw commented on issue #673: JENA-1812: Switch to Murmur3 for blank node allocation

[GitBox <gi...@apache.org>]

Claudenw commented on issue #673: JENA-1812: Switch to Murmur3 for blank node allocation
URL: https://github.com/apache/jena/pull/673#issuecomment-576153847
 
 
   There are 4 implementations of Murmur3:  There are 32 and 128 bit implementations both available in x86 and x64 variants.  The x86 and x64 variants are turned for different platforms but return different values.  The version available in commons-codec is murmur3-x86-128.
   
   The recent fix in commons-codec was a sign extension bug in seeds wherein if the seed was negative the conversion from int to long was not masked allowing the sign to extend across the upper bytes and changing the result with respect to the "C" code.  The name was also changed to reflect the actual hashing method being implemented.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@jena.apache.org
For additional commands, e-mail: pr-help@jena.apache.org