TheNeuralBit commented on a change in pull request #15588:
URL: https://github.com/apache/beam/pull/15588#discussion_r718871483
##########
File path: sdks/python/test-suites/tox/py38/build.gradle
##########
@@ -34,16 +34,11 @@ apply from: "../common.gradle"
// TODO(BEAM-8954): Remove this once tox uses isolated builds.
testPy38Cython.mustRunAfter testPython38, testPy38CloudCoverage
-toxTask "testPy38pyarrow-0", "py38-pyarrow-0"
-toxTask "testPy38pyarrow-1", "py38-pyarrow-1"
-toxTask "testPy38pyarrow-2", "py38-pyarrow-2"
-toxTask "testPy38pyarrow-3", "py38-pyarrow-3"
-toxTask "testPy38pyarrow-4", "py38-pyarrow-4"
-test.dependsOn "testPy38pyarrow-0"
-test.dependsOn "testPy38pyarrow-1"
-test.dependsOn "testPy38pyarrow-2"
-test.dependsOn "testPy38pyarrow-3"
-test.dependsOn "testPy38pyarrow-4"
+(0..5).each {version ->
Review comment:
Weird. I do see pyarrow-5 locally:
```
❯ ./gradlew :sdks:python:test-suites:tox:py38:test --dry-run
...
:sdks:python:test-suites:tox:py38:testPy38pyarrow-0 SKIPPED
:sdks:python:test-suites:tox:py38:testPy38pyarrow-1 SKIPPED
:sdks:python:test-suites:tox:py38:testPy38pyarrow-2 SKIPPED
:sdks:python:test-suites:tox:py38:testPy38pyarrow-3 SKIPPED
:sdks:python:test-suites:tox:py38:testPy38pyarrow-4 SKIPPED
:sdks:python:test-suites:tox:py38:testPy38pyarrow-5 SKIPPED
:sdks:python:test-suites:tox:py38:test SKIPPED
Deprecated Gradle features were used in this build, making it incompatible
with Gradle 7.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See
https://docs.gradle.org/6.9.1/userguide/command_line_interface.html#sec:command_line_warnings
BUILD SUCCESSFUL in 959ms
```
--
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]
