Details

    • Type: Sub-task Sub-task
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 5.5.0
    • Fix Version/s: 5.6.0
    • Component/s: Broker
    • Labels:
      None

      Description

      When using dynamic failover with randomize enabled, clients are not consistently rebalanced in an even manner. A test case is added that allows a minimum of 25% per broker on a three broker test and 40% distribution on a 2 broker test.

      1. AMQ-3706.patch
        16 kB
        Scott England-Sullivan

        Activity

        Hide
        Dejan Bosanac added a comment -

        svn revision 1311237 implements round-robin distribution of urls to clients instead of rnd. It should give a better distribution of clients over the cluster especially in small number of clients case.

        Show
        Dejan Bosanac added a comment - svn revision 1311237 implements round-robin distribution of urls to clients instead of rnd. It should give a better distribution of clients over the cluster especially in small number of clients case.
        Hide
        Dejan Bosanac added a comment -

        Fixed with svn revision 1242748

        The problem was that every client was responsible for randomizing his list of urls, and as every client uses it's own rnd generator we haven't had a uniform distribution.
        if the cluster rebalance is used, now we're doing a randomization on the broker side (always using same rnd generator) and clients now respect the url order broker supplied. It give much better distribution in this case.

        Show
        Dejan Bosanac added a comment - Fixed with svn revision 1242748 The problem was that every client was responsible for randomizing his list of urls, and as every client uses it's own rnd generator we haven't had a uniform distribution. if the cluster rebalance is used, now we're doing a randomization on the broker side (always using same rnd generator) and clients now respect the url order broker supplied. It give much better distribution in this case.
        Hide
        Scott England-Sullivan added a comment -

        Updated cluster test cases

        Show
        Scott England-Sullivan added a comment - Updated cluster test cases

          People

          • Assignee:
            Dejan Bosanac
            Reporter:
            Scott England-Sullivan
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development