From 864d94fe879a32de324da65a844e62a0260b222d Mon Sep 17 00:00:00 2001 From: Wenchen Fan <wenchen@databricks.com> Date: Fri, 2 Jun 2017 21:59:52 -0700 Subject: [PATCH] [SPARK-20974][BUILD] we should run REPL tests if SQL module has code changes ## What changes were proposed in this pull request? REPL module depends on SQL module, so we should run REPL tests if SQL module has code changes. ## How was this patch tested? N/A Author: Wenchen Fan <wenchen@databricks.com> Closes #18191 from cloud-fan/test. --- dev/run-tests.py | 2 +- dev/sparktestsupport/modules.py | 13 +++++++++++++ 2 files changed, 14 insertions(+), 1 deletion(-) diff --git a/dev/run-tests.py b/dev/run-tests.py index 818a0c9f48..72d148d7ea 100755 --- a/dev/run-tests.py +++ b/dev/run-tests.py @@ -111,7 +111,7 @@ def determine_modules_to_test(changed_modules): >>> x = [x.name for x in determine_modules_to_test([modules.sql])] >>> x # doctest: +NORMALIZE_WHITESPACE ['sql', 'hive', 'mllib', 'sql-kafka-0-10', 'examples', 'hive-thriftserver', - 'pyspark-sql', 'sparkr', 'pyspark-mllib', 'pyspark-ml'] + 'pyspark-sql', 'repl', 'sparkr', 'pyspark-mllib', 'pyspark-ml'] """ modules_to_test = set() for module in changed_modules: diff --git a/dev/sparktestsupport/modules.py b/dev/sparktestsupport/modules.py index 78b5b8b0f4..2971e0db40 100644 --- a/dev/sparktestsupport/modules.py +++ b/dev/sparktestsupport/modules.py @@ -123,6 +123,7 @@ sql = Module( ], ) + hive = Module( name="hive", dependencies=[sql], @@ -142,6 +143,18 @@ hive = Module( ) +repl = Module( + name="repl", + dependencies=[hive], + source_file_regexes=[ + "repl/", + ], + sbt_test_goals=[ + "repl/test", + ], +) + + hive_thriftserver = Module( name="hive-thriftserver", dependencies=[hive], -- GitLab