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

Break out StorableField from IndexableField



    • New


      In the field type branch we have strongly decoupled
      Document/Field/FieldType impl from the indexer, by having only a
      narrow API (IndexableField) passed to IndexWriter. This frees apps up
      use their own "documents" instead of the "user-space" impls we provide
      in oal.document.

      Similarly, with LUCENE-3309, we've done the same thing on the
      doc/field retrieval side (from IndexReader), with the

      But, maybe we should break out StorableField from IndexableField,
      such that when you index a doc you provide two Iterables – one for the
      IndexableFields and one for the StorableFields. Either can be null.

      One downside is possible perf hit for fields that are both indexed &
      stored (ie, we visit them twice, lookup their name in a hash twice,
      etc.). But the upside is a cleaner separation of concerns in API....


        1. LUCENE-3312-DocumentIterators-uwe.patch
          2 kB
          Uwe Schindler
        2. lucene-3312-patch-01.patch
          44 kB
          Nikola Tankovic
        3. lucene-3312-patch-02.patch
          68 kB
          Nikola Tankovic
        4. lucene-3312-patch-03.patch
          129 kB
          Nikola Tankovic
        5. lucene-3312-patch-04.patch
          126 kB
          Nikola Tankovic
        6. lucene-3312-patch-05.patch
          107 kB
          Nikola Tankovic
        7. lucene-3312-patch-06.patch
          109 kB
          Nikola Tankovic
        8. lucene-3312-patch-07.patch
          95 kB
          Nikola Tankovic
        9. lucene-3312-patch-08.patch
          103 kB
          Nikola Tankovic
        10. lucene-3312-patch-09.patch
          1 kB
          Nikola Tankovic
        11. lucene-3312-patch-10.patch
          9 kB
          Nikola Tankovic
        12. lucene-3312-patch-11.patch
          140 kB
          Nikola Tankovic
        13. lucene-3312-patch-12.patch
          141 kB
          Nikola Tankovic
        14. lucene-3312-patch-12a.patch
          147 kB
          Nikola Tankovic
        15. lucene-3312-patch-13.patch
          6 kB
          Nikola Tankovic
        16. lucene-3312-patch-14.patch
          3 kB
          Nikola Tankovic
        17. LUCENE-3312-reintegration.patch
          312 kB
          Uwe Schindler
        18. LUCENE-3312-reintegration.patch
          308 kB
          Uwe Schindler

        Issue Links



              ntankovic Nikola Tankovic
              mikemccand Michael McCandless
              2 Vote for this issue
              8 Start watching this issue