tkaymak commented on PR #40090: URL: https://github.com/apache/beam/pull/40090#issuecomment-5631379682
Thanks, all ten addressed in 58303d6. The registry split is `PipelineTranslatorCommon` with a thin `PipelineTranslatorBatch`, as suggested. One more commit is folded in, 75dbe36: the red Spark Versions run on this PR was `cancel()` stopping the SparkSession while the execution thread was still translating. In that fork it hit the cold static init of `PipelineTranslatorBatch` and poisoned the JVM for 12 batch tests. `cancel()` now waits for the execution thread before the session stop, with a unit test. Pre-existing in the shared runner, it surfaced because #40093 made the module run in CI again. -- 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]
