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

factor out a TopGroupsShardResponseProcessor.fillResultIds method

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

Details

    • Task
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • None
    • 8.2, 9.0
    • None
    • None

    Description

      The TopGroupsShardResponseProcessor.process method e.g. #L54-L215 does quite a few things and factoring out a fillResultIds (or similarly named) method for the logically distinct #L192-L214 portion could help with code comprehension.

      Attachments

        Activity

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

          People

            cpoerschke Christine Poerschke
            cpoerschke Christine Poerschke
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment