[GitHub] [helix] kaisun2000 opened a new issue #1216: Identify and fix leak of ClusterManager

[GitBox <gi...@apache.org>]

kaisun2000 opened a new issue #1216:
URL: https://github.com/apache/helix/issues/1216


   There are cases of leaking ClusterManager and its subclasses. The leaked threads can be in hundreds, Note, relying on GC of the ClusterManager object does not mean the thread is shutdown. For example TestNoDoubleAssign. 


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



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For additional commands, e-mail: reviews-help@helix.apache.org

[GitHub] [helix] lei-xia commented on issue #1216: Identify and fix leak of ClusterManager

[GitBox <gi...@apache.org>]

lei-xia commented on issue #1216:
URL: https://github.com/apache/helix/issues/1216#issuecomment-669007222


   Can we make this issue/description more clear here? i.e, this is to fix the leak of ClusterManager in our unit tests.  Otherwise, it could create confusion here that we may have thread leak issue in our HelixManager code.


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



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For additional commands, e-mail: reviews-help@helix.apache.org