drpmma commented on code in PR #7665:
URL: https://github.com/apache/rocketmq/pull/7665#discussion_r1427593336
##########
broker/src/main/java/org/apache/rocketmq/broker/processor/PopMessageProcessor.java:
##########
@@ -493,8 +509,22 @@ private CompletableFuture<Long> popMsgFromQueue(String
attemptId, boolean isRetr
PopMessageRequestHeader requestHeader, int queueId, long restNum, int
reviveQid,
Channel channel, long popTime, ExpressionMessageFilter messageFilter,
StringBuilder startOffsetInfo,
StringBuilder msgOffsetInfo, StringBuilder orderCountInfo) {
- String topic = isRetry ?
KeyBuilder.buildPopRetryTopic(requestHeader.getTopic(),
- requestHeader.getConsumerGroup()) : requestHeader.getTopic();
+ return popMsgFromQueue(attemptId, isRetry, getMessageResult,
requestHeader, queueId, restNum, reviveQid, channel,
+ popTime, messageFilter, startOffsetInfo, msgOffsetInfo,
orderCountInfo, true);
+ }
+
+ private CompletableFuture<Long> popMsgFromQueue(String attemptId, boolean
isRetry, GetMessageResult getMessageResult,
+ PopMessageRequestHeader requestHeader, int queueId, long restNum, int
reviveQid,
+ Channel channel, long popTime, ExpressionMessageFilter messageFilter,
StringBuilder startOffsetInfo,
+ StringBuilder msgOffsetInfo, StringBuilder orderCountInfo, boolean
isRetryV2) {
+ String topic;
Review Comment:
fixed
--
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: commits-unsubscr...@rocketmq.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
