Aias00 opened a new pull request, #10771: URL: https://github.com/apache/rocketmq/pull/10771
### What changed This PR makes Proxy gRPC `RouteActivity#queryAssignment` skip queue rows that do not have a master broker entry in the broker map. Previously, the code passed a possibly-null master `Broker` into the protobuf `MessageQueue` builder, which could fail the request exceptionally when route metadata only had non-master broker ids. ### Why Malformed or partial route metadata should produce a controlled assignment response. If no valid assignment can be built after skipping invalid rows, the existing `FORBIDDEN` / `no readable queue` response is returned. Fixes #10770. ### Verification ```bash mvn -pl proxy -Dtest=RouteActivityTest test ``` Result: BUILD SUCCESS. `RouteActivityTest` ran 8 tests with 0 failures/errors/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]
