Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 7.0
    • None
    • None
    • New

    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.

      Michael McCandless 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)

      Attachments

        1. LUCENE_7500_avoid_leafReader_fields.patch
          54 kB
          David Smiley
        2. LUCENE_7500_avoid_leafReader_fields.patch
          42 kB
          David Smiley
        3. LUCENE_7500_Remove_LeafReader_fields.patch
          44 kB
          David Smiley
        4. LUCENE_7500_Remove_LeafReader_fields.patch
          44 kB
          David Smiley
        5. LUCENE_7500_Remove_LeafReader_fields.patch
          41 kB
          David Smiley

        Issue Links

          Activity

            People

              dsmiley David Smiley
              dsmiley David Smiley
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Slack

                  Issue deployment