[LUCENE-10373] Can we optimize TaxonomyFacetSumIntAssociations#sumValues?

• Type: Improvement
• Status: Open
• Priority: Minor
• Resolution: Unresolved
• Component/s: modules/facet
• Lucene Fields: New
• Assignee: Unassigned
• Reporter: Greg Miller
• Created: 11/Jan/22 23:38
• Updated: 28/Aug/22 16:34

Description

It looks like TaxonomyFacetSumIntAssociations#sumValues might also benefit from reduced null checks just like in LUCENE-10350 (doing away with the use of the parent class increment method).

While we're at it, it would be nice to test moving to little endian encoding of the ord/value since we're trying to deprecate BitUtil.VH_BE_INT.

The real challenge is that we don't have benchmark tasks for this class. Maybe we should add benchmarking first? I opened a luceneutil issue for this: https://github.com/mikemccand/luceneutil/issues/158