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 1bee07e39f1b [SPARK-46855][INFRA] Add `sketch` to the dependencies of 
the `catalyst` in `module.py`
1bee07e39f1b is described below

commit 1bee07e39f1b5aef6ce81e028207691f1dd1fc7c
Author: yangjie01 <[email protected]>
AuthorDate: Thu Jan 25 08:26:13 2024 -0800

    [SPARK-46855][INFRA] Add `sketch` to the dependencies of the `catalyst` in 
`module.py`
    
    ### What changes were proposed in this pull request?
    This pr add `sketch` to the dependencies of the `catalyst` module in 
`module.py` due to `sketch` is direct dependency of `catalyst` module.
    
    ### Why are the changes needed?
    Ensure that when modifying the `sketch` module, both `catalyst` and 
cascading modules will trigger tests.
    
    ### 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 #44878 from LuciferYang/SPARK-46855.
    
    Lead-authored-by: yangjie01 <[email protected]>
    Co-authored-by: YangJie <[email protected]>
    Signed-off-by: Dongjoon Hyun <[email protected]>
---
 dev/sparktestsupport/modules.py | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/dev/sparktestsupport/modules.py b/dev/sparktestsupport/modules.py
index be3e798b0779..b9541c4be9b3 100644
--- a/dev/sparktestsupport/modules.py
+++ b/dev/sparktestsupport/modules.py
@@ -179,6 +179,15 @@ launcher = Module(
     ],
 )
 
+sketch = Module(
+    name="sketch",
+    dependencies=[tags],
+    source_file_regexes=[
+        "common/sketch/",
+    ],
+    sbt_test_goals=["sketch/test"],
+)
+
 core = Module(
     name="core",
     dependencies=[kvstore, network_common, network_shuffle, unsafe, launcher, 
utils],
@@ -200,7 +209,7 @@ api = Module(
 
 catalyst = Module(
     name="catalyst",
-    dependencies=[tags, core, api],
+    dependencies=[tags, sketch, core, api],
     source_file_regexes=[
         "sql/catalyst/",
     ],
@@ -315,15 +324,6 @@ connect = Module(
     ],
 )
 
-sketch = Module(
-    name="sketch",
-    dependencies=[tags],
-    source_file_regexes=[
-        "common/sketch/",
-    ],
-    sbt_test_goals=["sketch/test"],
-)
-
 graphx = Module(
     name="graphx",
     dependencies=[tags, core],


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to