qqu0127 commented on PR #2011: URL: https://github.com/apache/helix/pull/2011#issuecomment-1087807581
Adding more ideas and discussion from offline regarding the fundamental problem statement: - @qqu0127 : I’d like to first check with you, do you agree if it’s a code issue we need to fix? (compared to disabling / removing the test). I’m debating myself with another argument like -- it is there for a long time, it seems not hurting anything and almost a feature. - @narendly : Short answer is that I do think we need to have coverage of this ... We have had issues for Helix where metrics were never properly cleaned up and caused JVM heap OOM issues -- 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]
