[LUCENE-8921] IndexSearcher.termStatistics should not require TermStates but docFreq and totalTermFreq

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 8.1
• Fix Version/s: 8.3
• Component/s: core/search
• Lucene Fields: New
• Assignee: David Smiley
• Reporter: Bruno Roustant
• Created: 17/Jul/19 08:03
• Updated: 22/Sep/22 17:41
• Resolved: 17/Sep/19 20:56

Description

IndexSearcher.termStatistics(Term term, TermStates context) is the way to create a TermStatistics. It requires a TermStates param although it only cares about the docFreq and totalTermFreq.

 

For customizations that what to create TermStatistics based on docFreq and totalTermFreq, but that do not have available TermStates, this method forces to create a TermStates instance (which is not very lightweight) only to pass two ints.

termStatistics could be modified to the following signature:

termStatistics(Term term, int docFreq, int totalTermFreq)

Since it would change the API, it could be done in master for next major release.

Issue Links

• links to: GitHub Pull Request #797