[
https://issues.apache.org/jira/browse/LUCENE-5604?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13967866#comment-13967866
]
Uwe Schindler commented on LUCENE-5604:
---------------------------------------
bq. The JVM recognizes pairs of shifts that amount to a rotate and replaces
them with an intrinsic.
I still think we should replace them by the "methods". This is the same like
replacing the ternary {{? :}} with {{Number.compare(x,y)}} for comparators.
Brings no improvements, just better readability in Java 7 and is less
error-prone (cf. the possible overflows if implementing the compare with a dumb
ternary op).
> Should we switch BytesRefHash to MurmurHash3?
> ---------------------------------------------
>
> Key: LUCENE-5604
> URL: https://issues.apache.org/jira/browse/LUCENE-5604
> Project: Lucene - Core
> Issue Type: Improvement
> Components: core/index
> Reporter: Michael McCandless
> Assignee: Michael McCandless
> Fix For: 4.9, 5.0
>
> Attachments: BytesRefHash.perturb.patch, LUCENE-5604.patch
>
>
> MurmurHash3 has better hashing distribution than the current hash function we
> use for BytesRefHash which is a simple multiplicative function with 31
> multiplier (same as Java's String.hashCode, but applied to bytes not chars).
> Maybe we should switch ...
--
This message was sent by Atlassian JIRA
(v6.2#6252)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
