Solr
  1. Solr
  2. SOLR-6208

JettySolrRunner QueuedThreadPool's configuration code is never executed

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.10, 6.0
    • Component/s: None
    • Labels:
      None

      Description

      QueuedThreadPool's configuration in the current code is effectively dead (pools are initialized after server's startup, not upon instantiation).

        Activity

        Hide
        Dawid Weiss added a comment -

        Patch attached.

        Show
        Dawid Weiss added a comment - Patch attached.
        Hide
        ASF subversion and git services added a comment -

        Commit 1610408 from shalin@apache.org in branch 'dev/trunk'
        [ https://svn.apache.org/r1610408 ]

        SOLR-6208: JettySolrRunner QueuedThreadPool's configuration code is never executed

        Show
        ASF subversion and git services added a comment - Commit 1610408 from shalin@apache.org in branch 'dev/trunk' [ https://svn.apache.org/r1610408 ] SOLR-6208 : JettySolrRunner QueuedThreadPool's configuration code is never executed
        Hide
        ASF subversion and git services added a comment -

        Commit 1610410 from shalin@apache.org in branch 'dev/branches/branch_4x'
        [ https://svn.apache.org/r1610410 ]

        SOLR-6208: JettySolrRunner QueuedThreadPool's configuration code is never executed

        Show
        ASF subversion and git services added a comment - Commit 1610410 from shalin@apache.org in branch 'dev/branches/branch_4x' [ https://svn.apache.org/r1610410 ] SOLR-6208 : JettySolrRunner QueuedThreadPool's configuration code is never executed
        Hide
        Shalin Shekhar Mangar added a comment -

        Thanks for catching this Dawid!

        I hope you don't mind that I committed your patch.

        Show
        Shalin Shekhar Mangar added a comment - Thanks for catching this Dawid! I hope you don't mind that I committed your patch.
        Hide
        Dawid Weiss added a comment -

        No, no, not at all. Thanks Shalin!

        Show
        Dawid Weiss added a comment - No, no, not at all. Thanks Shalin!

          People

          • Assignee:
            Dawid Weiss
            Reporter:
            Dawid Weiss
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development