Uploaded image for project: 'Lucene - Core'
  1. Lucene - Core
  2. LUCENE-4524

Merge DocsEnum and DocsAndPositionsEnum into PostingsEnum

    Details

    • Lucene Fields:
      New

      Description

      spinnoff from http://www.gossamer-threads.com/lists/lucene/java-dev/172261

      hey folks, 
      
      I have spend a hell lot of time on the positions branch to make 
      positions and offsets working on all queries if needed. The one thing 
      that bugged me the most is the distinction between DocsEnum and 
      DocsAndPositionsEnum. Really when you look at it closer DocsEnum is a 
      DocsAndFreqsEnum and if we omit Freqs we should return a DocIdSetIter. 
      Same is true for 
      DocsAndPostionsAndPayloadsAndOffsets*YourFancyFeatureHere*Enum. I 
      don't really see the benefits from this. We should rather make the 
      interface simple and call it something like PostingsEnum where you 
      have to specify flags on the TermsIterator and if we can't provide the 
      sufficient enum we throw an exception? 
      I just want to bring up the idea here since it might simplify a lot 
      for users as well for us when improving our positions / offset etc. 
      support. 
      
      thoughts? Ideas? 
      
      simon 
      

        Attachments

        1. LUCENE-4524.patch
          210 kB
          Simon Willnauer
        2. LUCENE-4524.patch
          148 kB
          Simon Willnauer
        3. LUCENE-4524.patch
          548 kB
          Alan Woodward
        4. LUCENE-4524.patch
          755 kB
          Alan Woodward
        5. LUCENE-4524.patch
          758 kB
          Alan Woodward
        6. LUCENE-4524.patch
          721 kB
          Alan Woodward

          Issue Links

            Activity

              People

              • Assignee:
                romseygeek Alan Woodward
                Reporter:
                simonw Simon Willnauer
              • Votes:
                2 Vote for this issue
                Watchers:
                8 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: