RockteMQ-AI commented on issue #10945:
URL: https://github.com/apache/rocketmq/issues/10945#issuecomment-5303058726

   **Issue Evaluation**
   
   Category: `bug` | Status: **Confirmed**
   
   The reported index misalignment in `PopConsumerService` has been verified 
against the current `develop` branch.
   
   **Root Cause:** In `PopConsumerService.java` (line ~434-436), the loop 
iterates `getMessageResultList` by index `i` and looks up 
`popConsumerRecordList.get(i)` with the same index. When a single fetch result 
contains multiple messages, the record list has one entry per message while the 
result list has one entry per fetch — so after a multi-message batch, index `i` 
points into the wrong batch.
   
   **Impact:** With `popResponseReturnActualRetryTopic` enabled, a retry result 
can be misidentified as normal and skip retry-message recoding, leading to 
incorrect message routing.
   
   **Severity:** High — affects POP consumption correctness under multi-message 
batch scenarios.
   
   **Verified at:** 
`broker/src/main/java/org/apache/rocketmq/broker/pop/PopConsumerService.java:434-446`
   
   An automated fix proposal can be generated. Reply `/approve` to proceed with 
PR generation.
   
   ---
   *Automated evaluation by RockteMQ-AI*


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