lostluck commented on PR #29829: URL: https://github.com/apache/beam/pull/29829#issuecomment-1871421754
Great question! Fixing that is out of scope for this PR. integration.go does not *start* runners. The Go SDK has never auto-started runners, largely due to that being an undocumented expectation that has never been fulfilled. The lack of runner lead to deadline failure, and then retuning a nil Pipeline results. The test in particular doesn't fail out when there are nil PipelineResults though, so the metrics check then fails. This could be fixed by not returning a nil pipeline results in such cases in the default runner lib, or by making the test check if a nil was returned. -- 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]
