You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by mx...@apache.org on 2018/11/30 11:15:55 UTC
[beam] 05/07: [BEAM-6146] Add precommit for portable python
This is an automated email from the ASF dual-hosted git repository.
mxm pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/beam.git
commit 49c8386b48b2e57b38041ace7824132b8806bc08
Author: Ankur Goenka <an...@gmail.com>
AuthorDate: Thu Nov 29 19:06:08 2018 -0800
[BEAM-6146] Add precommit for portable python
---
sdks/python/build.gradle | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/sdks/python/build.gradle b/sdks/python/build.gradle
index 7c54239..aba3904 100644
--- a/sdks/python/build.gradle
+++ b/sdks/python/build.gradle
@@ -161,6 +161,12 @@ task preCommit() {
dependsOn "lint"
}
+task portablePreCommit() {
+ dependsOn ':beam-runners-flink_2.11-job-server-container:docker'
+ dependsOn ':beam-sdks-python-container:docker'
+ dependsOn 'portableWordCount'
+}
+
/*************************************************************************************************/
// E2E integration testing and validates runner testing
@@ -283,6 +289,9 @@ task portableWordCount(dependsOn: 'installGcpTest') {
}
}
}
+// Make sure that the job server is built before the portableWordCount is executed.
+portableWordCount.mustRunAfter ':beam-runners-flink_2.11-job-server-container:docker'
+portableWordCount.mustRunAfter ':beam-sdks-python-container:docker'
// Run single or a set of integration tests with provided test options and pipeline options.
task integrationTest(dependsOn: ['installGcpTest', 'sdist']) {