Github user danny0405 commented on a diff in the pull request:

    https://github.com/apache/storm/pull/2433#discussion_r152716651
  
    --- Diff: storm-client/src/jvm/org/apache/storm/daemon/worker/Worker.java 
---
    @@ -382,6 +374,33 @@ public void establishLogSettingCallback() {
             workerState.stormClusterState.topologyLogConfig(topologyId, 
this::checkLogConfigChanged);
         }
     
    +    /**
    +     * Send a heartbeat to local supervisor first to check if supervisor 
is ok for heartbeating.
    +     */
    +    private void heartbeatToMasterIfLocalbeatFail(LSWorkerHeartbeat 
lsWorkerHeartbeat) {
    --- End diff --
    
    Normally we use local-state and supervisor reporting heartbeats, only if 
when supervisor goes down, worker will reports heartbeats directly.


---

Reply via email to