You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@ratis.apache.org by GitBox <gi...@apache.org> on 2020/05/06 05:52:41 UTC

[GitHub] [incubator-ratis] runzhiwang opened a new pull request #90: RATIS-925. Fix Memory leak of RaftServerImpl for no remove from static RaftServerMetrics::metricsMap

runzhiwang opened a new pull request #90:
URL: https://github.com/apache/incubator-ratis/pull/90


   **What's the problem ?**
   As the image shows, there are 2066 instances of RaftServerImpl, most of them are Closed, and should be GC, but actually not.
   ![image](https://user-images.githubusercontent.com/51938049/81141951-bc9a7580-8fa0-11ea-9516-f0fb38067113.png)
   
   **What's the reason ?**
   You can find from the image 2040 RaftServerImpl were held by RaftServerMetrics::metricsMap which is a staic map 
   ![image](https://user-images.githubusercontent.com/51938049/81141965-c7eda100-8fa0-11ea-95f7-841981576bca.png)
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org