remove useless code in ReflectUtils.java

[thisiswanghy]

Remove useless code

What is the purpose of the change

The findMethodByMethodSignature method in ReflectUtils.java，The signature field is not used

Brief changelog

Verifying this change

Checklist

• Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
• Each commit in the pull request should have a meaningful subject line and body.
• Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
• Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
• Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
• Add some description to dubbo-website project if you are requesting to add a feature.
• GitHub Actions works fine on your own branch.
• If this contribution is large, please follow the Software Donation Guide.

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (a8b8382) 69.56% compared to head (46404d6) 69.53%.

Additional details and impacted files

@@             Coverage Diff              @@
##                3.2   #13382      +/-   ##
============================================
- Coverage     69.56%   69.53%   -0.04%     
  Complexity        2        2              
============================================
  Files          1653     1653              
  Lines         71623    71620       -3     
  Branches      10269    10267       -2     
============================================
- Hits          49827    49801      -26     
- Misses        17095    17119      +24     
+ Partials       4701     4700       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[thisiswanghy]

I looked at the commit history for this class and found that the pr #8684 removes all caching inside ReflectUtils.java, leaving behind the processing of the signature field.

But I don't understand why the cache was removed, it seems to me that it should be kept. I would like to understand the thinking behind removing the cache？

[AlbumenJ]

I looked at the commit history for this class and found that the pr #8684 removes all caching inside ReflectUtils.java, leaving behind the processing of the signature field.

But I don't understand why the cache was removed, it seems to me that it should be kept. I would like to understand the thinking behind removing the cache？

The cache is global. This will result a reference from Dubbo's classes (usually are GC root) to actual classes. And these classes can not be released after classloader is unused.