Uploaded image for project: 'Apache RocketMQ'
  1. Apache RocketMQ
  2. ROCKETMQ-140

Register higher version broker against old name servers

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 4.1.0-incubating
    • Fix Version/s: 4.1.0-incubating
    • Component/s: rocketmq-namesrv
    • Labels:
      None

      Description

      When register higher version brokers again old name servers, IndexOutOfBoundaryException may be thrown, causing registration failure.

        Issue Links

          Activity

          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user coveralls commented on the issue:

          https://github.com/apache/incubator-rocketmq/pull/77

          [![Coverage Status](https://coveralls.io/builds/10559397/badge)](https://coveralls.io/builds/10559397)

          Coverage increased (+2.8%) to 33.856% when pulling *e41ff32628253c410b28712f3e16dc98fc737679 on lizhanhui:ROCKETMQ-140* into *d7decc84abc32dab63ee423d4d904f28d18cb1d7 on apache:develop*.

          Show
          githubbot ASF GitHub Bot added a comment - Github user coveralls commented on the issue: https://github.com/apache/incubator-rocketmq/pull/77 [! [Coverage Status] ( https://coveralls.io/builds/10559397/badge)](https://coveralls.io/builds/10559397 ) Coverage increased (+2.8%) to 33.856% when pulling * e41ff32628253c410b28712f3e16dc98fc737679 on lizhanhui: ROCKETMQ-140 * into * d7decc84abc32dab63ee423d4d904f28d18cb1d7 on apache:develop *.
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user coveralls commented on the issue:

          https://github.com/apache/incubator-rocketmq/pull/77

          [![Coverage Status](https://coveralls.io/builds/10559397/badge)](https://coveralls.io/builds/10559397)

          Coverage increased (+2.8%) to 33.856% when pulling *e41ff32628253c410b28712f3e16dc98fc737679 on lizhanhui:ROCKETMQ-140* into *d7decc84abc32dab63ee423d4d904f28d18cb1d7 on apache:develop*.

          Show
          githubbot ASF GitHub Bot added a comment - Github user coveralls commented on the issue: https://github.com/apache/incubator-rocketmq/pull/77 [! [Coverage Status] ( https://coveralls.io/builds/10559397/badge)](https://coveralls.io/builds/10559397 ) Coverage increased (+2.8%) to 33.856% when pulling * e41ff32628253c410b28712f3e16dc98fc737679 on lizhanhui: ROCKETMQ-140 * into * d7decc84abc32dab63ee423d4d904f28d18cb1d7 on apache:develop *.
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user coveralls commented on the issue:

          https://github.com/apache/incubator-rocketmq/pull/77

          [![Coverage Status](https://coveralls.io/builds/10559397/badge)](https://coveralls.io/builds/10559397)

          Coverage increased (+2.8%) to 33.856% when pulling *e41ff32628253c410b28712f3e16dc98fc737679 on lizhanhui:ROCKETMQ-140* into *d7decc84abc32dab63ee423d4d904f28d18cb1d7 on apache:develop*.

          Show
          githubbot ASF GitHub Bot added a comment - Github user coveralls commented on the issue: https://github.com/apache/incubator-rocketmq/pull/77 [! [Coverage Status] ( https://coveralls.io/builds/10559397/badge)](https://coveralls.io/builds/10559397 ) Coverage increased (+2.8%) to 33.856% when pulling * e41ff32628253c410b28712f3e16dc98fc737679 on lizhanhui: ROCKETMQ-140 * into * d7decc84abc32dab63ee423d4d904f28d18cb1d7 on apache:develop *.
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user Jaskey commented on the issue:

          https://github.com/apache/incubator-rocketmq/pull/77

          This issue exits for long, whose is trival but has great impact when managing rocketmq in console, which may result in the cluster page not able to render.

          So this pr should be reviewed and merged ASAP.

          Show
          githubbot ASF GitHub Bot added a comment - Github user Jaskey commented on the issue: https://github.com/apache/incubator-rocketmq/pull/77 This issue exits for long, whose is trival but has great impact when managing rocketmq in console, which may result in the cluster page not able to render. So this pr should be reviewed and merged ASAP.
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user lizhanhui commented on the issue:

          https://github.com/apache/incubator-rocketmq/pull/77

          As this PR is pretty trivial, I'll merge it soon.

          Show
          githubbot ASF GitHub Bot added a comment - Github user lizhanhui commented on the issue: https://github.com/apache/incubator-rocketmq/pull/77 As this PR is pretty trivial, I'll merge it soon.
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user lizhanhui commented on the issue:

          https://github.com/apache/incubator-rocketmq/pull/77

          Merged.

          Show
          githubbot ASF GitHub Bot added a comment - Github user lizhanhui commented on the issue: https://github.com/apache/incubator-rocketmq/pull/77 Merged.
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user lizhanhui closed the pull request at:

          https://github.com/apache/incubator-rocketmq/pull/77

          Show
          githubbot ASF GitHub Bot added a comment - Github user lizhanhui closed the pull request at: https://github.com/apache/incubator-rocketmq/pull/77
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user vongosling commented on the issue:

          https://github.com/apache/incubator-rocketmq/pull/77

          Sorry to response for this PR. It seems good ~

          Show
          githubbot ASF GitHub Bot added a comment - Github user vongosling commented on the issue: https://github.com/apache/incubator-rocketmq/pull/77 Sorry to response for this PR. It seems good ~

            People

            • Assignee:
              lizhanhui Zhanhui Li
              Reporter:
              lizhanhui Zhanhui Li
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development