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

   <!-- Please make sure the target branch is right. In most case, the target 
branch should be `develop`. -->
   
   ### Which Issue(s) This PR Fixes
   
   - Fixes #10425
   
   ### Brief Description
   
   This reverts #9521 and restores the topic config `DataVersion` advancement 
when `enableSplitRegistration` is enabled.
   
   With split broker registration, a broker sends topic configs to the name 
server in multiple registration requests. The name server records the broker 
`DataVersion` after each request and uses it to decide whether existing topic 
route data should be updated. If all split requests carry the same 
`DataVersion`, later batches can be treated as unchanged for topics that 
already exist under the broker.
   
   Restoring the per-wrapper `DataVersion` advancement keeps each split 
registration batch visible as a distinct update. This PR also adds a focused 
regression test for `TopicConfigManager#buildSerializeWrapper`.
   
   ### How Did You Test This Change?
   
   - `/usr/local/bin/mvn -pl broker -am -Dtest=TopicConfigManagerTest 
-DfailIfNoTests=false -DskipITs -DskipCheckStyle test`
   


-- 
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