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

   ### What changed
   
   - Reject null or empty handle lists in 
`ClusterMessageService.batchAckMessage()` before resolving broker address from 
the first handle.
   - Return a failed future with `INVALID_RECEIPT_HANDLE` instead of throwing 
`IndexOutOfBoundsException`.
   - Add regression coverage for null and empty handle lists and verify the MQ 
client is not called.
   
   Fixes #10780.
   
   ### Verification
   
   `mvn -pl proxy -Dtest=ClusterMessageServiceTest test`
   
   Result: BUILD SUCCESS. `ClusterMessageServiceTest` ran 3 tests with 0 
failures, errors, or skips. Checkstyle and SpotBugs also passed in the Maven 
run.
   


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