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

Fencepost error in LeaderInitiatedRecoveryThread

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

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 4.10.2, 5.0
    • None
    • None

    Description

      At line 106:

          while (continueTrying && ++tries < maxTries) {
      

      should be

          while (continueTrying && ++tries <= maxTries) {
      

      This is only a problem when called from DistributedUpdateProcessor, as it can have maxTries set to 1, which means the loop is never actually run.

      Attachments

        1. SOLR-6511.patch
          11 kB
          Timothy Potter
        2. SOLR-6511.patch
          29 kB
          Timothy Potter

        Activity

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

          People

            thelabdude Timothy Potter
            romseygeek Alan Woodward
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment