scwhittle commented on PR #33317: URL: https://github.com/apache/beam/pull/33317#issuecomment-2536082294
I'm not sure what the guarantees are on detecting errors with the channel if there are not keep-alives and the channel is otherwise idle. https://github.com/grpc/proposal/blob/master/A8-client-side-keepalive.md#design-background indicates that tcp keepalives can end up taking hours to detect issues. Since we're just on a local connection here perhaps that doesn't apply for some reason though. And if that is the case we could expect the runner to eventually shutdown the sdk as unresponsive. -- 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]
