GitHub user zhan849 opened a pull request:
https://github.com/apache/helix/pull/162
[HELIX-683] clean monitoring cache upon helix controller enable monitoring
In this PR I added methods to clear monitoring records in cache when we
enable cluster status monitoring. I also added tests to reproduce situation
that a resource missed top state, controller lost leadership, resource regain
top state, controller regain leadership, which will cause a metrics reporting
problem
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/zhan849/helix
harry/controller-monitor-cache-cleanup
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/helix/pull/162.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #162
----
commit 373da77547fa1ea4a39c760e80da75e9d453d4f5
Author: Harry Zhang <zhan849@...>
Date: 2018-03-26T19:14:07Z
[HELIX-683] clean monitoring cache upon helix controller enable monitoring
----
---
