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

Delete solr/core/src/test-files/solr/configsets/_default

    XMLWordPrintableJSON

Details

    • Test
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 8.5, 9.0
    • None
    • None

    Description

      For as long as we've had a _default configset in solr, we've also had a copy of that default in core/src/test-files/ - as well as a unit test that confirms they are identical.

      It's never really been clear to me why we have this duplication, instead of just having the test-framework take the necessary steps to ensure that server/solr/configsets/_default is properly used when running tests.

      I'd like to propose we eliminate the duplication since it only ever seems to cause problems (notably spurious test failures when people modify the _default configset w/o remembering that they need to make identical edits to the test-files clone) and instead have SolrTestCase set the (already existing & supported) solr.default.confdir system property to point to the (already existing) ExternalPaths.DEFAULT_CONFIGSET

      Attachments

        1. SOLR-13794.patch
          250 kB
          Chris M. Hostetter
        2. SOLR-13794.patch
          243 kB
          Chris M. Hostetter
        3. SOLR-13794_code_only.patch
          13 kB
          Chris M. Hostetter
        4. SOLR-13794_code_only.patch
          13 kB
          Chris M. Hostetter

        Issue Links

          Activity

            People

              hossman Chris M. Hostetter
              hossman Chris M. Hostetter
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: