[spark] branch master updated: [SPARK-35483][FOLLOWUP][TESTS] Enable docker_integration_tests for catalyst/sql module changes too

[dongjoon]

This is an automated email from the ASF dual-hosted git repository.

dongjoon pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/master by this push:
     new 57896e6  [SPARK-35483][FOLLOWUP][TESTS] Enable 
docker_integration_tests for catalyst/sql module changes too
57896e6 is described below

commit 57896e662e9bd16cd2400afa53cea7981f42b714
Author: Dongjoon Hyun <dongj...@apache.org>
AuthorDate: Mon Jun 28 22:59:56 2021 -0700

    [SPARK-35483][FOLLOWUP][TESTS] Enable docker_integration_tests for 
catalyst/sql module changes too
    
    ### What changes were proposed in this pull request?
    
    This PR aims to enable `docker_integration_tests` when `catalyst` and `sql` 
module changes additionally.
    
    ### Why are the changes needed?
    
    Currently, `catalyst` and `sql` module changes do not trigger the JDBC 
integration test.
    
    ### Does this PR introduce _any_ user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    N/A
    
    Closes #33125 from dongjoon-hyun/SPARK-35483.
    
    Authored-by: Dongjoon Hyun <dongj...@apache.org>
    Signed-off-by: Dongjoon Hyun <dongj...@apache.org>
---
 dev/sparktestsupport/modules.py | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/dev/sparktestsupport/modules.py b/dev/sparktestsupport/modules.py
index 5d28b9e..2ba3390 100644
--- a/dev/sparktestsupport/modules.py
+++ b/dev/sparktestsupport/modules.py
@@ -177,6 +177,9 @@ catalyst = Module(
     sbt_test_goals=[
         "catalyst/test",
     ],
+    environ=None if "GITHUB_ACTIONS" not in os.environ else {
+        "ENABLE_DOCKER_INTEGRATION_TESTS": "1"
+    },
 )
 
 sql = Module(
@@ -188,6 +191,9 @@ sql = Module(
     sbt_test_goals=[
         "sql/test",
     ],
+    environ=None if "GITHUB_ACTIONS" not in os.environ else {
+        "ENABLE_DOCKER_INTEGRATION_TESTS": "1"
+    },
 )
 
 hive = Module(
@@ -769,7 +775,7 @@ spark_ganglia_lgpl = Module(
 
 docker_integration_tests = Module(
     name="docker-integration-tests",
-    dependencies=[],
+    dependencies=[sql],
     build_profile_flags=["-Pdocker-integration-tests"],
     source_file_regexes=["external/docker-integration-tests"],
     sbt_test_goals=["docker-integration-tests/test"],

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org