Skip to content

Commit ef336ad

Browse files
committed
[SPARK-51948][BUILD] Make changes of mllib module to trigger the test for the connect module
### What changes were proposed in this pull request? This pr appends `mllib` to the `dependencies` list of the `connect` module in the file `dev/sparktestsupport/modules.py`, ensuring that changes to the `mllib` module will trigger the tests for the `connect` module. ### Why are the changes needed? Changes to the `mllib` module should trigger the tests for the connect module. ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? - Pass Github Actions ### Was this patch authored or co-authored using generative AI tooling? No Closes #50753 from LuciferYang/connect-mllib. Lead-authored-by: yangjie01 <yangjie01@baidu.com> Co-authored-by: YangJie <yangjie01@baidu.com> Signed-off-by: yangjie01 <yangjie01@baidu.com>
1 parent 64a7e6d commit ef336ad

File tree

1 file changed

+11
-12
lines changed

1 file changed

+11
-12
lines changed

dev/sparktestsupport/modules.py

Lines changed: 11 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -329,18 +329,6 @@ def __hash__(self):
329329
],
330330
)
331331

332-
connect = Module(
333-
name="connect",
334-
dependencies=[hive, avro, protobuf],
335-
source_file_regexes=[
336-
"sql/connect",
337-
],
338-
sbt_test_goals=[
339-
"connect/test",
340-
"connect-client-jvm/test",
341-
],
342-
)
343-
344332
graphx = Module(
345333
name="graphx",
346334
dependencies=[tags, core],
@@ -420,6 +408,17 @@ def __hash__(self):
420408
],
421409
)
422410

411+
connect = Module(
412+
name="connect",
413+
dependencies=[hive, avro, protobuf, mllib],
414+
source_file_regexes=[
415+
"sql/connect",
416+
],
417+
sbt_test_goals=[
418+
"connect/test",
419+
"connect-client-jvm/test",
420+
],
421+
)
423422

424423
examples = Module(
425424
name="examples",

0 commit comments

Comments
 (0)