Uploaded image for project: 'Solr'
  1. Solr
  2. SOLR-12625

Combine SolrDocumentFetcher and RetrieveFieldsOptimizer

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 7.5, 8.0
    • None
    • None

    Description

      We have SolrDocumentFetcher and RetrieveFieldsOptimizer. The
      relationship between the two is unclear at first glance. Using
      SolrDocumentFetcher by itself is (or can be) inefficient.

      WDYT about combining the two? Is there a good reason you would want to
      use SolrDocumentFetcher instead of RetrieveFieldsOptimizer?

      Ideally I'd want to be able to write code like:

      solrDocumentFetcher.fillDocValuesMostEfficiently

      That created an optimizer and "did the right thing".

      Assigning to myself to keep track, but if anyone feels motivated feel free to take it over.

      Attachments

        1. SOLR-12625.patch
          57 kB
          Erick Erickson
        2. SOLR-12625.patch
          49 kB
          Erick Erickson
        3. SOLR-12625.patch
          47 kB
          Erick Erickson
        4. SOLR-12625.patch
          20 kB
          Erick Erickson
        5. SOLR-12625.patch
          31 kB
          Erick Erickson

        Issue Links

          is related to

          Improvement - An improvement or enhancement to an existing feature or task. SOLR-12598 Do not fetch non-stored fields

          • Major - Major loss of function.
          • Closed
          relates to

          Improvement - An improvement or enhancement to an existing feature or task. SOLR-10816 Change uniqueKey to use docValues and not stored field

          • Major - Major loss of function.
          • Open

          Improvement - An improvement or enhancement to an existing feature or task. SOLR-12596 DocValues page should explain terms like SORTED_SET, SORTED_NUMERIC

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Closed

          Activity

            People

              erickerickson Erick Erickson
              erickerickson Erick Erickson
              Votes:
              3 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: