Zhengcy05 opened a new pull request, #16359: URL: https://github.com/apache/dubbo/pull/16359
## What is the purpose of the change? This PR makes `DescriptorUtils#findReflectionMethodDescriptor` return directly from the generic branch, matching the existing echo branch. fixs #16330 The change is a small consistency cleanup in `dubbo-rpc-triple`. It does not intend to change behavior. The generic and echo paths both resolve a single cached descriptor, so the generic branch can return immediately instead of assigning to a local variable and falling through to the final return. ## Checklist - [x] Make sure there is a [[GitHub_issue](https://github.com/apache/dubbo/issues)](https://github.com/apache/dubbo/issues) field for the change. - [x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why. - [x] Write necessary unit-test to verify your logic correction. No new unit test was added because this is a behavior-preserving refactor. - [x] Make sure gitHub actions can pass. Local verification was attempted with `mvn -pl dubbo-rpc/dubbo-rpc-triple -am -DskipTests compile`, but the build was blocked by an existing `dubbo-maven-plugin` plugin descriptor issue (`No plugin descriptor found at META-INF/maven/plugin.xml`), which is unrelated to this change. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
