yuluo-yx opened a new pull request, #10866: URL: https://github.com/apache/rocketmq/pull/10866
### Which Issue(s) This PR Fixes - Fixes #10861 ### Brief Description `CleanControllerBrokerMetaSubCommand` built a lazy stream to parse controller broker IDs but never consumed it, so invalid values were not validated. This change eagerly parses every semicolon-separated ID before any admin request. A regression test supplies `1;not-a-number` and verifies that command execution rejects it with `IllegalArgumentException`. ### How Did You Test This Change? - `mise exec [email protected]+10 -- mvn -pl tools -am -DskipITs -Dtest=CleanControllerBrokerMetaSubCommandTest -Dsurefire.failIfNoSpecifiedTests=false -Dspotbugs.skip=true test` - Result: `BUILD SUCCESS`; 1 test, 0 failures, 0 errors, 0 skipped. - Scope check: 2 files, 48 changed lines. -- 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]
