[LUCENE-7500] Remove LeafReader.fields()

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 7.0
• Lucene Fields: New
• Assignee: David Smiley
• Reporter: David Smiley
• Created: 17/Oct/16 13:57
• Updated: 28/Aug/22 15:04
• Resolved: 15/Jun/17 05:02

Description

Fields seems like a pointless intermediary between the LeafReader and Terms. Why not have LeafReader.getTerms(fieldName) instead? One loses the ability to get the count and iterate over indexed fields, but it's not clear what real use-cases are for that and such rare needs could figure that out with FieldInfos.

mikemccand pointed out that we'd probably need to re-introduce a TermVectors class since TV's are row-oriented not column-oriented. IMO they should be column-oriented but that'd be a separate issue.

(p.s. I'm lacking time to do this w/i the next couple months so if someone else wants to tackle it then great)

Issue Links

• relates to: LUCENE-7633 Rename Terms to IndexedField (was to FieldTerms)
• requires: LUCENE-7876 Avoid needless calls to LeafReader.fields and MultiFields.getFields