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

   ### Summary
   
   - Skip lite-client offline cleanup when settings have no subscription 
section or no subscription entries.
   - Avoid indexing `subscriptions(0)` for incomplete lite-consumer settings.
   - Add a regression test for lite consumers without subscription entries.
   
   ### Motivation
   
   Closes #10681.
   
   `GrpcClientSettingsManager.offlineClientLiteSubscription` previously assumed 
lite-consumer settings always contain at least one subscription entry. 
Incomplete settings could throw `IndexOutOfBoundsException` and fall into the 
broad error log path, including the full settings object.
   
   ### Tests
   
   - `mvn -pl proxy -Dtest=GrpcClientSettingsManagerTest -DfailIfNoTests=false 
test`
   
   Result: BUILD SUCCESS; 8 tests passed; checkstyle reported 0 violations. The 
run emits existing JaCoCo instrumentation warnings under the local JDK, and one 
existing test intentionally logs a simulated sync failure, but tests and build 
completed successfully.
   


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