lxorc commented on PR #13838:
URL: 
https://github.com/apache/dolphinscheduler/pull/13838#issuecomment-1491214786

   > We don't need to add this check, this case might be due to we registering 
an empty node? Could you please describe how to reproduce this.
   
   Yes, by default a `worker` node will be loaded in 
`registryClient.subscribe(REGISTRY_DOLPHINSCHEDULER_WORKERS, new 
WorkerDataListener()); `
   
   ```java
   registryClient.subscribe(REGISTRY_DOLPHINSCHEDULER_WORKERS, new 
WorkerDataListener()); 
   String[] parts = path.split("/");  // REGISTRY_DOLPHINSCHEDULER_WORKERS = 
/nodes/worker
   final String workerAddress = parts[parts.length - 1];  // workerAddress  = 
worker
   syncSingleWorkerNodeInfo(workerAddress, JSONUtils.parseObject(data, 
WorkerHeartBeat.class));
   workerNodeInfo.put(workerAddress, info);  // put "worker" key into the 
workerNodeInfo
   ` ` `


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

Reply via email to