Uploaded image for project: 'ActiveMQ .Net'
  1. ActiveMQ .Net
  2. AMQNET-285

Error in FailoverTransport.BuildBackups()

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

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 1.4.1
    • NMS
    • None

    Description

      Around line 1148 in FailoverTransport.cs, there is the following code:

      if(backups.Count < BackupPoolSize)
      {
      break;
      }

      In the Java code in FailoverTransport.java, there is a loop that has the opposite effect as the C# code.
      for (Iterator<URI> iter = connectList.iterator(); iter.hasNext() && backups.size() < backupPoolSize {

      In Java, we STAY IN the loop as long as we haven't filled up the backups collection. In C#, we're incorrectly BAILING OUT if we haven't filled up the collection.

      Attachments

        Activity

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

          People

            tabish Timothy A. Bish
            muamw10 Andrew Westberg
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment