junkaixue commented on code in PR #2344:
URL: https://github.com/apache/helix/pull/2344#discussion_r1095335668
##########
helix-core/src/main/java/org/apache/helix/monitoring/mbeans/ClusterStatusMonitor.java:
##########
@@ -55,6 +55,81 @@
import org.slf4j.LoggerFactory;
public class ClusterStatusMonitor implements ClusterStatusMonitorMBean {
+ private class AsyncMissingTopStateMonitor extends Thread {
Review Comment:
This will be meaningful if our pipeline speed is very fast. Otherwise, it
will not help as the cache updated state is refreshed per pipeline. I dont
believe we need build this thread but just rely on pipeline call as we did
before.
But we need to remove the constraint of doing only final reporting.
We can discuss about it tomorrow f2f.
--
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]
