Skip to content

Commit c8bbab6

Browse files
committed
[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. (cherry picked from commit 864d94f) Signed-off-by: Wenchen Fan <wenchen@databricks.com>
1 parent 478874e commit c8bbab6

File tree

2 files changed

+14
-1
lines changed

2 files changed

+14
-1
lines changed

dev/run-tests.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ def determine_modules_to_test(changed_modules):
111111
>>> x = [x.name for x in determine_modules_to_test([modules.sql])]
112112
>>> x # doctest: +NORMALIZE_WHITESPACE
113113
['sql', 'hive', 'mllib', 'sql-kafka-0-10', 'examples', 'hive-thriftserver',
114-
'pyspark-sql', 'sparkr', 'pyspark-mllib', 'pyspark-ml']
114+
'pyspark-sql', 'repl', 'sparkr', 'pyspark-mllib', 'pyspark-ml']
115115
"""
116116
modules_to_test = set()
117117
for module in changed_modules:

dev/sparktestsupport/modules.py

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -123,6 +123,7 @@ def __hash__(self):
123123
],
124124
)
125125

126+
126127
hive = Module(
127128
name="hive",
128129
dependencies=[sql],
@@ -142,6 +143,18 @@ def __hash__(self):
142143
)
143144

144145

146+
repl = Module(
147+
name="repl",
148+
dependencies=[hive],
149+
source_file_regexes=[
150+
"repl/",
151+
],
152+
sbt_test_goals=[
153+
"repl/test",
154+
],
155+
)
156+
157+
145158
hive_thriftserver = Module(
146159
name="hive-thriftserver",
147160
dependencies=[hive],

0 commit comments

Comments
 (0)