[
https://issues.apache.org/jira/browse/BEAM-6777?focusedWorklogId=258178&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-258178
]
ASF GitHub Bot logged work on BEAM-6777:
----------------------------------------
Author: ASF GitHub Bot
Created on: 11/Jun/19 22:20
Start Date: 11/Jun/19 22:20
Worklog Time Spent: 10m
Work Description: robinyqiu commented on pull request #8827: [BEAM-6777]
Let HealthzServlet respond actual health information of SDK harnesses
URL: https://github.com/apache/beam/pull/8827#discussion_r292682650
##########
File path:
runners/google-cloud-dataflow-java/worker/src/main/java/org/apache/beam/runners/dataflow/worker/SdkHarnessRegistries.java
##########
@@ -107,6 +108,17 @@ public void unregisterWorkerClient(FnApiControlClient
controlClient) {
workers.remove(worker);
}
LOG.info("Unregistered Control client {}", worker != null ?
worker.getWorkerId() : null);
+
+ // unregisterWorkerClient() will be called only when the connection
between SDK harness and
+ // runner harness is broken or SDK harness respond to runner harness
with an error. In either
+ // case, the SDK should be marked as unhealthy.
+ sdkHarnessesAreHealthy.set(false);
Review comment:
This doesn't work for all cases now. It only covers cases where the grpc
channel between SDK harness and runner harness is broken/erroneous.
I am looking at what changes needed to be added to the SDK harness side now
(This PR only covers changes on the runner harness side), in order to make it
cover more cases. I plan to send another PR to do that, or patch this PR if
there aren't many changes needed.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Issue Time Tracking
-------------------
Worklog Id: (was: 258178)
Time Spent: 5h 40m (was: 5.5h)
> SDK Harness Resilience
> ----------------------
>
> Key: BEAM-6777
> URL: https://issues.apache.org/jira/browse/BEAM-6777
> Project: Beam
> Issue Type: Improvement
> Components: runner-dataflow
> Reporter: Sam Rohde
> Assignee: Yueyang Qiu
> Priority: Major
> Time Spent: 5h 40m
> Remaining Estimate: 0h
>
> If the Python SDK Harness crashes in any way (user code exception, OOM, etc)
> the job will hang and waste resources. The fix is to add a daemon in the SDK
> Harness and Runner Harness to communicate with Dataflow to restart the VM
> when stuckness is detected.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
