AlbumenJ commented on PR #13382: URL: https://github.com/apache/dubbo/pull/13382#issuecomment-1818146027
> 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. -- 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]
