Uploaded image for project: 'Solr'
  1. Solr
  2. SOLR-8975 SolrClient setters should be deprecated in favor of Solr Client Builder methods
  3. SOLR-16583

Rename CloudSolrClientBuilder in SolrTestCaseJ4 to RandomizingCloudSolrClientBuilder

    XMLWordPrintableJSON

Details

    • Sub-task
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 9.0
    • main (10.0), 9.2
    • Integration Tests
    • None

    Description

      I've been chasing may way through the confusion of various Builders, and to my dismay discovered that we have in SolrTestCaseJ4 defined a builder that is designed to do some randomization!

      public static class CloudSolrClientBuilder extends CloudLegacySolrClient.Builder

       

      This gets super confusing cause there is a test called CloudSolrClientBuilderTest that makes me think it tests the above class, but it doesnt'.   in a previous commit we renamed a similar confusing pattern to

      RandomizingCloudHttp2SolrClientBuilder

      so this would be to clarify what this builder is.

        

      Attachments

        Issue Links

          Activity

            People

              epugh Eric Pugh
              epugh Eric Pugh
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 20m
                  20m