apache / lucene

Apache Lucene open-source search software
https://lucene.apache.org/
Apache License 2.0
2.46k stars 979 forks source link

Simplify bytes arrays in NumericLeafComparator to long #13246

Closed gf2121 closed 2 months ago

gf2121 commented 3 months ago

Simplify complex byte array comparisons as long comparison.

gf2121 commented 3 months ago

Out of curiosity, are you seeing a speedup from this change?

On wikimedium10m, this patch is slightly faster with little P-value:

                            TaskQPS baseline      StdDevQPS my_modified_version      StdDev                Pct diff p-value
                      TermDTSort      275.90      (1.2%)      278.55      (0.8%)    1.0% (  -1% -    3%) 0.003
           HighTermDayOfYearSort      459.59      (1.4%)      465.90      (1.1%)    1.4% (  -1% -    3%) 0.000