ibzib commented on pull request #14995: URL: https://github.com/apache/beam/pull/14995#issuecomment-859726183
LGTM, thanks for fixing this. > I did notice that Spark PVR Streaming tests have been broken into two tests in #14483 but not for Batch tests, are we going to apply the same improvement for batch mode tests ? I don't think there's much of a reason to separate them, since unlike the streaming tests both Spark versions consistently pass tests (while in streaming, Spark 3 is much less flaky than Spark 2, for unknown reasons). -- 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. For queries about this service, please contact Infrastructure at: [email protected]
