Solr
  1. Solr
  2. SOLR-2225

CoreContainer#register should use checkDefault to normalize the core name

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.0-ALPHA, 5.0
    • Component/s: multicore
    • Labels:
      None

      Description

      fail case:

      start with default collection set to collection1
      remove core collection1
      default collection on CoreContainer is still set to collection1
      add core collection1
      it doesn't act like the default core

      we might do as the summary suggests, or when the default core is removed, we reset to no default core until one is again explicitly set

        Issue Links

          Activity

          Hide
          Hoss Man added a comment -

          i didn't dig into this, but based on the description of the problem i would bet money it was fixed by the changes in SOLR-2605.

          Show
          Hoss Man added a comment - i didn't dig into this, but based on the description of the problem i would bet money it was fixed by the changes in SOLR-2605 .
          Hide
          Mark Miller added a comment -

          I've updated this test and somehow this passes now - another issue must have solved it. I'll just commit the new test.

          Show
          Mark Miller added a comment - I've updated this test and somehow this passes now - another issue must have solved it. I'll just commit the new test.
          Hide
          Robert Muir added a comment -

          3.4 -> 3.5

          Show
          Robert Muir added a comment - 3.4 -> 3.5
          Hide
          Robert Muir added a comment -

          Bulk move 3.2 -> 3.3

          Show
          Robert Muir added a comment - Bulk move 3.2 -> 3.3
          Hide
          Mark Miller added a comment -

          Fix + test.

          Show
          Mark Miller added a comment - Fix + test.
          Hide
          Robert Muir added a comment -

          There isn't any patch here yet, can we move out to 3.2?

          Show
          Robert Muir added a comment - There isn't any patch here yet, can we move out to 3.2?
          Hide
          Mark Miller added a comment -

          I don't remember this at all - digging in though.

          Show
          Mark Miller added a comment - I don't remember this at all - digging in though.

            People

            • Assignee:
              Mark Miller
              Reporter:
              Mark Miller
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development