Solr
  1. Solr
  2. SOLR-3379

Make logging of threadpool configuration report correct values

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 3.6, 4.0-ALPHA
    • Fix Version/s: 3.6.1, 4.0-ALPHA
    • Component/s: None
    • Labels:
      None

      Description

      SOLR-3221 allows configuration of the shard handler thread pool. When the thread pool configuration is changed the configuration changed is logged but the value reported is always the socket timeout.
      This patch will report the value of the configuration item changed.

      1. SOLR-3379-3x_branch.patch
        0.6 kB
        David M. Lyle
      2. SOLR-3379-trunk.patch
        0.6 kB
        David M. Lyle

        Activity

        Hide
        Hoss Man added a comment -

        Sami: Could you please triage this for 4.0? commit if you think it's ready, otherwise remove the fix version?

        Show
        Hoss Man added a comment - Sami: Could you please triage this for 4.0? commit if you think it's ready, otherwise remove the fix version?
        Hide
        Sami Siren added a comment -

        I missed this issue completely earlier, sorry about that. I merged my earlier commit from 4.x (1332160) so this is now fixed also in 3.6.1 (1351779).

        Thanks David for reporting the problem and also providing the fix!

        Show
        Sami Siren added a comment - I missed this issue completely earlier, sorry about that. I merged my earlier commit from 4.x (1332160) so this is now fixed also in 3.6.1 (1351779). Thanks David for reporting the problem and also providing the fix!
        Hide
        Greg Bowyer added a comment -

        Nice catch I cant believe I made that mistake

        Show
        Greg Bowyer added a comment - Nice catch I cant believe I made that mistake
        Hide
        Uwe Schindler added a comment -

        Bulk close for 3.6.1

        Show
        Uwe Schindler added a comment - Bulk close for 3.6.1

          People

          • Assignee:
            Sami Siren
            Reporter:
            David M. Lyle
          • Votes:
            1 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development