[
https://issues.apache.org/jira/browse/BEAM-12419?focusedWorklogId=605393&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-605393
]
ASF GitHub Bot logged work on BEAM-12419:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 02/Jun/21 17:40
Start Date: 02/Jun/21 17:40
Worklog Time Spent: 10m
Work Description: kw2542 commented on a change in pull request #14923:
URL: https://github.com/apache/beam/pull/14923#discussion_r644185526
##########
File path:
runners/java-fn-execution/src/main/java/org/apache/beam/runners/fnexecution/environment/ExternalEnvironmentFactory.java
##########
@@ -113,6 +113,7 @@ public RemoteEnvironment createEnvironment(Environment
environment, String worke
ManagedChannelFactory.createDefault().forDescriptor(externalPayload.getEndpoint());
BeamFnApi.StartWorkerResponse startWorkerResponse =
BeamFnExternalWorkerPoolGrpc.newBlockingStub(managedChannel)
+ .withWaitForReady()
Review comment:
Good point, it should be separated out in a different PR at least.
My thought on this is similar as the job server request that when runner
connects to external service, it is possible that external service may not be
up and running yet if we bring up external service together with the main job.
In addition, there may be intermittent errors, therefore, `withWaitForReady`
seems to be a good candidate here.
I also think that we could provide customizable deadline/timeout here too
which requires changes to external_payload model.
What are your thoughts on this?
--
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:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 605393)
Time Spent: 1.5h (was: 1h 20m)
> Timeout/WaitForReady support in java gRPC calls
> -----------------------------------------------
>
> Key: BEAM-12419
> URL: https://issues.apache.org/jira/browse/BEAM-12419
> Project: Beam
> Issue Type: Bug
> Components: runner-core
> Reporter: Ke Wu
> Assignee: Ke Wu
> Priority: P2
> Time Spent: 1.5h
> Remaining Estimate: 0h
>
> gRPC calls in java Portable pipeline does not have retry or timeout enabled,
> unlike python BEAM-7933
> Email discussion:
> https://lists.apache.org/thread.html/rd4de8d54b22777913a8e20ee04e8b9d5c9aa7c16942ab36d7d168425%40%3Cdev.beam.apache.org%3E
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
