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

SegmentReader.getFieldNames ignores FieldOption.DOC_VALUES

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 4.0-ALPHA
    • 4.0-ALPHA
    • None
    • None
    • New

    Description

      we use this getFieldNames api in segmentmerger if we merge something that isn't a SegmentReader (e.g. FilterIndexReader)

      it looks to me that if you use a FilterIndexReader, call addIndexes(Reader...) the docvalues will be simply dropped.

      I dont think its enough to just note that the field has docvalues either right? We need to also set the type
      correctly in the merged field infos? This would imply that instead of FieldOption.DOCVALUES, we need to have a
      FieldOption for each ValueType so that we correctly update the type.

      But looking at FI.update/setDocValues, it doesn't look like we 'type-promote' here anyway?

      Attachments

        1. LUCENE-3623_test.patch
          3 kB
          Robert Muir
        2. LUCENE-3623.patch
          6 kB
          Robert Muir
        3. LUCENE-3623.patch
          6 kB
          Robert Muir
        4. LUCENE-3623.patch
          6 kB
          Robert Muir

        Activity

          People

            Unassigned Unassigned
            rcmuir Robert Muir
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: