ibzib commented on a change in pull request #15321:
URL: https://github.com/apache/beam/pull/15321#discussion_r687272877
##########
File path: sdks/go/test/run_validatesrunner_tests.sh
##########
@@ -79,6 +79,9 @@ RUNNER=portable
# packages are executed in parallel.
TIMEOUT=1h
+# Default limit on simultaneous test binaries/packages being executed.
+PARALLEL=3
Review comment:
I saw in the docs that the default is the number of cores, so it should
be fine to leave it unset for the Dataflow runner, which doesn't need to do a
lot of heavy-duty computation locally. But for Flink I think it should be 1. If
nothing else, it makes reading the logs easier -- otherwise the logs for
different jobs get interleaved and it becomes very difficult to tell which logs
came from which test.
--
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]
