zjncs opened a new pull request, #11135:
URL: https://github.com/apache/rocketmq/pull/11135

   ### Motivation
   
   `mqadmin exportMetadata` summed `retryQueueNums` of the same subscription 
group over every master of the cluster, the way topic queue numbers are 
totaled. But a subscription group config is cluster-wide: a group exported from 
an N-broker cluster came out with N times its real `retryQueueNums`, and 
re-importing that metadata inflated every group's retry queues.
   
   ### Modifications
   
   Keep the first seen config per group instead of summing, matching the 
cluster-wide semantics of subscription group config.
   
   ### Verification
   
   Fail-before (new test, run against the unpatched code):
   
   ```
   
ExportMetadataCommandTest#testSubscriptionGroupRetryQueueNumsNotAccumulatedAcrossBrokers
   java.lang.AssertionError: retryQueueNums must not be summed across the two 
masters expected:<1> but was:<2>
   ```
   
   Pass-after:
   
   ```
   Tests run: 1, Failures: 0, Errors: 0, Skipped: 0 -- ExportMetadataCommandTest
   ```
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to