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

Merge 'pre-register', 'create' and 'register' core methods in CoreContainer

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

Details

    • Improvement
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • 4.8.1, 6.0
    • 4.10
    • None
    • None

    Description

      We have a bunch of duplicated logic in CoreContainer startup, CoreAdminHandler core creation and core tests that always calls the same three methods, 'preRegisterInZk', 'create' and 'register'. These should really all be in one API call.

      Attachments

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

          People

            romseygeek Alan Woodward
            romseygeek Alan Woodward
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment