wang-jiahua opened a new pull request, #10565:
URL: https://github.com/apache/rocketmq/pull/10565
### Which Issue(s) This PR Fixes
Fixes #9633
### Brief Description
V1 retry topic naming uses underscore as separator (`%RETRY%{cid}_{topic}`),
which is ambiguous because both topic and group names may contain underscores.
This causes different (group, topic) pairs to map to the same retry topic,
leading to cross-topic consumption.
V2 naming (using `+` separator) already exists but was disabled by default.
This PR changes `enableRetryTopicV2` default from false to true.
Existing V1 retry topics are still readable because
`retrieveMessageFromPopRetryTopicV1` remains true (backward compatible).
### How Did You Test This Change
Unit tests: `KeyBuilderTest` (3 new tests demonstrating V1 collision and V2
correctness).
--
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]
