You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@rocketmq.apache.org by GitBox <gi...@apache.org> on 2026/08/25 04:57:04 UTC
[GH] (rocketmq/fix/enable-retry-topic-v2-by-default): Workflow run "Run Integration Tests" failed!
The GitHub Actions job "Run Integration Tests" on rocketmq.git/fix/enable-retry-topic-v2-by-default has failed.
Run started by GitHub user wang-jiahua (triggered by wang-jiahua).
Head commit for run:
7d4da0fb357777f363348357448a657f791c9ad5 / wangjiahua.wjh <wa...@alibaba-inc.com>
[ISSUE #9633] Enable retry topic V2 by default to prevent name collision
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 and potential data leakage.
V2 naming (using '+' separator) already exists but was disabled by default.
This commit changes enableRetryTopicV2 default from false to true.
Existing V1 retry topics are still readable because
retrieveMessageFromPopRetryTopicV1 remains true (backward compatible).
Changes:
- BrokerConfig: enableRetryTopicV2 default false -> true
- KeyBuilderTest: 3 new tests demonstrating V1 collision and V2 correctness
Report URL: https://github.com/apache/rocketmq/actions/runs/32809217345
With regards,
GitHub Actions via GitBox