tvalentyn commented on a change in pull request #12239:
URL: https://github.com/apache/beam/pull/12239#discussion_r467346745
##########
File path: build.gradle
##########
@@ -220,13 +220,12 @@ task pythonPreCommit() {
dependsOn ":sdks:python:test-suites:tox:py36:preCommitPy36"
dependsOn ":sdks:python:test-suites:tox:py37:preCommitPy37"
dependsOn ":sdks:python:test-suites:tox:py38:preCommitPy38"
- dependsOn ":sdks:python:test-suites:dataflow:py2:preCommitIT"
- dependsOn ":sdks:python:test-suites:dataflow:py2:preCommitIT_V2"
- dependsOn ":sdks:python:test-suites:dataflow:py37:preCommitIT"
- dependsOn ":sdks:python:test-suites:dataflow:py37:preCommitIT_V2"
- // We don't include Py35, Py36 precommit ITs to reduce quota footprint.
- // We can reconsider if we ever see an issue that these suites would
- // have caught. Note that the same tests will still run in postcommit.
+ dependsOn ":sdks:python:test-suites:dataflow:preCommitIT"
+ dependsOn ":sdks:python:test-suites:dataflow:preCommitIT_V2"
+ // We don't include all supported Python versions precommit ITs to
Review comment:
We can remove this comment.
##########
File path: build.gradle
##########
@@ -248,9 +247,9 @@ task pythonFormatterPreCommit() {
task python2PostCommit() {
dependsOn
":sdks:python:test-suites:portable:py2:crossLanguagePythonJavaKafkaIOFlink"
- dependsOn ":sdks:python:test-suites:portable:py2:crossLanguageTests"
+ dependsOn ":sdks:python:test-suites:portable:crossLanguageTests"
Review comment:
I'm not sure I follow this change - this is currently still a python2
PostCommit suite, so it should include py2 only tests.
However you could convert the targets that are not version-specific, such as
```
task portablePythonPreCommit()
task pythonSparkPostCommit()
```
to point to portable/build.gradle
----------------------------------------------------------------
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]
