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

Provide default for ConcurrentUpdateSolrClient's "queueSize" param

Agile BoardAttach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 8.0
    • 7.2, 8.0
    • SolrJ
    • None

    Description

      A user on the mailing list recently pointed out that if it's not specified explicitly as a Builder option, ConcurrentUpdateSolrClient will default to using a queueSize of 0. This value gets passed in to the underlying queue data structure which throws an IllegalArgumentException, with an error message that isn't obvious to those unfamiliar with the internals.

      We should provide a better default than the uninitialized-variable default of 0. Almost all occurrences in the code fall between 5 and 10, so a queueSize in that range should be uncontroversial.

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            anshum Anshum Gupta
            gerlowskija Jason Gerlowski
            Votes:
            1 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment