WangzJi commented on PR #3023: URL: https://github.com/apache/dubbo-go/pull/3023#issuecomment-3290705110
> I see that this pr allows extension to unregister, but extension should be registered once later, and will not be created frequently, which will not lead to memory leakage. Can you answer my question? Hi @marsevilspirit, Sorry for the late response. You make a very valid point, in typical usage, extensions are registered once and won't be created frequently, so they shouldn't cause memory leaks. <img width="893" height="670" alt="image" src="https://github.com/user-attachments/assets/14c4c6a8-9686-455a-95f5-819fa07b46c0"; /> Added the unregistration functionality mainly to address the issue mentioned in #3008 : global extension cache lacks expiration cleanup mechanism. While the impact is limited under current usage patterns, this is a preventive fix that provides foundational support for potential dynamic extension scenarios. The implementation cost is very low, and we've also added `memory_leak_test.go` to verify the functionality works correctly. -- 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: notifications-unsubscr...@dubbo.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@dubbo.apache.org For additional commands, e-mail: notifications-h...@dubbo.apache.org
