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

SolrIndexConfig parameter in SolrIndexSearcher constructor is not used

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 4.10
    • 5.0
    • None
    • None

    Description

      The following constructor of SolrIndexSearcher doesn't use 'config'.

      SolrIndexSearcher
      public SolrIndexSearcher(SolrCore core, String path, IndexSchema schema, SolrIndexConfig config, String name, DirectoryReader r, boolean closeReader, boolean enableCache, boolean reserveDirectory, DirectoryFactory directoryFactory)
      

      It doesn't make sense to pass in the SolrIndexConfig when we're passing in the DirectoryReader (and asserting that it's never null). Prior to LUCENE-5666, when 'r' was null, the config was used to get a reader but not any more.

      I'll just remove the param from the constructor and remove it from all places that calls this version of the constructor.

      Attachments

        1. SOLR-6597.patch
          4 kB
          Anshum Gupta

        Activity

          People

            anshum Anshum Gupta
            anshum Anshum Gupta
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: