Re: Patch to change murmurhash implementation slightly

2023-04-24 Thread Patrick Zhai
I did a quick run with your patch, but since I turned on the CMS as well as TieredMergePolicy I'm not sure how fair the comparison is. Here's the result: Candidate: Indexer: indexing done (890209 msec); total 2620 docs Indexer: waitForMerges done (71622 msec) Indexer: finished (961877 msec)

Patch to change murmurhash implementation slightly

2023-04-24 Thread Thomas Dullien
Hey all, I've been experimenting with fixing some low-hanging performance fruit in the ElasticSearch codebase, and came across the fact that the MurmurHash implementation that is used by ByteRef.hashCode() is reading 4 bytes per loop iteration (which is likely an artifact from 32-bit