(spark) branch branch-3.5 updated: [SPARK-46855][INFRA][3.5] Add `sketch` to the dependencies of the `catalyst` in `module.py`

[dongjoon]

This is an automated email from the ASF dual-hosted git repository.

dongjoon pushed a commit to branch branch-3.5
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/branch-3.5 by this push:
     new e5a654e818b4 [SPARK-46855][INFRA][3.5] Add `sketch` to the 
dependencies of the `catalyst` in `module.py`
e5a654e818b4 is described below

commit e5a654e818b4698260807a081e5cf3d71480ac13
Author: yangjie01 <yangji...@baidu.com>
AuthorDate: Thu Jan 25 22:35:38 2024 -0800

    [SPARK-46855][INFRA][3.5] 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 #44893 from LuciferYang/SPARK-46855-35.
    
    Authored-by: yangjie01 <yangji...@baidu.com>
    Signed-off-by: Dongjoon Hyun <dongj...@apache.org>
---
 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 33d253a47ea0..d29fc8726018 100644
--- a/dev/sparktestsupport/modules.py
+++ b/dev/sparktestsupport/modules.py
@@ -168,6 +168,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],
@@ -181,7 +190,7 @@ core = Module(
 
 catalyst = Module(
     name="catalyst",
-    dependencies=[tags, core],
+    dependencies=[tags, sketch, core],
     source_file_regexes=[
         "sql/catalyst/",
     ],
@@ -295,15 +304,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: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org