From d711b52673273a59526523458f534665ec8620aa Mon Sep 17 00:00:00 2001 From: Wenchen Fan Date: Fri, 2 Jun 2017 16:18:05 -0700 Subject: [PATCH 1/2] we should run REPL tests if SQL core has code changes --- dev/sparktestsupport/modules.py | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/dev/sparktestsupport/modules.py b/dev/sparktestsupport/modules.py index 78b5b8b0f4b5..2971e0db4049 100644 --- a/dev/sparktestsupport/modules.py +++ b/dev/sparktestsupport/modules.py @@ -123,6 +123,7 @@ def __hash__(self): ], ) + hive = Module( name="hive", dependencies=[sql], @@ -142,6 +143,18 @@ def __hash__(self): ) +repl = Module( + name="repl", + dependencies=[hive], + source_file_regexes=[ + "repl/", + ], + sbt_test_goals=[ + "repl/test", + ], +) + + hive_thriftserver = Module( name="hive-thriftserver", dependencies=[hive], From 3a1d26cbf1602bf44f0f13fc56eea2e7c317107e Mon Sep 17 00:00:00 2001 From: Wenchen Fan Date: Fri, 2 Jun 2017 16:31:33 -0700 Subject: [PATCH 2/2] fix test --- dev/run-tests.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dev/run-tests.py b/dev/run-tests.py index 818a0c9f4841..72d148d7ea0f 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: