damondouglas commented on PR #32040: URL: https://github.com/apache/beam/pull/32040#issuecomment-2263520415
@Abacn this is different in that the runner needs to launch an underlying Job Management service written in a different language and shut it down gracefully. The reason for introducing this PR is that relying on existing tools in Beam do not properly shutdown the gRPC channels, the code is too nested to safely refactor. This has caused errors in relying on proper shutdown, even when calling waitUntilFinish. The result is that the underlying process remains running without owndership and the only way to end the process is to use `lsof`, for example followed by `kill`. Please review this assuming it is 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
