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']) {