[
https://issues.apache.org/jira/browse/BEAM-14157?focusedWorklogId=752105&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-752105
]
ASF GitHub Bot logged work on BEAM-14157:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 04/Apr/22 07:11
Start Date: 04/Apr/22 07:11
Worklog Time Spent: 10m
Work Description: arunpandianp commented on code in PR #17191:
URL: https://github.com/apache/beam/pull/17191#discussion_r841426413
##########
runners/google-cloud-dataflow-java/worker/src/main/java/org/apache/beam/runners/dataflow/worker/windmill/GrpcWindmillServer.java:
##########
@@ -632,6 +632,8 @@ private static long uniqueId() {
// The following should be protected by synchronizing on this, except for
// the atomics which may be read atomically for status pages.
private StreamObserver<RequestT> requestObserver;
+ // Indicates if the current stream in requestObserver is client closed
Review Comment:
done, reworded.
Issue Time Tracking
-------------------
Worklog Id: (was: 752105)
Time Spent: 3h (was: 2h 50m)
> Don't send requests on a closed windmill Grpc streams
> -----------------------------------------------------
>
> Key: BEAM-14157
> URL: https://issues.apache.org/jira/browse/BEAM-14157
> Project: Beam
> Issue Type: Bug
> Components: runner-dataflow
> Reporter: Arun Pandian
> Assignee: Arun Pandian
> Priority: P2
> Time Spent: 3h
> Remaining Estimate: 0h
>
> GrpcWindmillServer could send requests requests on client closed streams.
> This leads to windmill streams getting stalling occasionally for few seconds
> to few minutes. grpc-java doc says not to call onNext to send after a stream
> is client closed.
> When the streams get stalled it is logged as "Output channel stalled for {}s,
> outbound thread {}." from
> [here|https://github.com/apache/beam/blob/7727dc99ed5dc1fb46166ef496ab3607ee2779f8/runners/google-cloud-dataflow-java/worker/src/main/java/org/apache/beam/runners/dataflow/worker/windmill/DirectStreamObserver.java#L100]
> Ref:
> [https://github.com/grpc/grpc-java/blob/master/stub/src/main/java/io/grpc/stub/StreamObserver.java#L62]
> [https://github.com/apache/beam/blob/master/runners/google-cloud-dataflow-java/worker/src/main/java/org/apache/beam/runners/dataflow/worker/windmill/GrpcWindmillServer.java#L939]
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
