lostluck commented on a change in pull request #15321:
URL: https://github.com/apache/beam/pull/15321#discussion_r687289262
##########
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 think the main source of conflict here is that I didn't see that Spark
and Samza were already being hard limited to 1 test binary at a time anyway.
I'm keeping the 3 limit for Dataflow and Portable, and have made it flaggable.
Also changed things so they're only specifying the flag once, what I had would
override it.
A later PR might change this kind of hacky overrides to happen at the script
invocation level, but I'm currently making these changes via the web UI, and
would rather keep it simple.
--
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]
