Skip to content

Commit

Permalink
[SPARK-46855][INFRA][3.4] Add sketch to the dependencies of the `ca…
Browse files Browse the repository at this point in the history
…talyst` 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 #44894 from LuciferYang/SPARK-46855-34.

Authored-by: yangjie01 <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
  • Loading branch information
LuciferYang authored and dongjoon-hyun committed Jan 26, 2024
1 parent 3130ac9 commit 441c33d
Showing 1 changed file with 10 additions and 10 deletions.
20 changes: 10 additions & 10 deletions dev/sparktestsupport/modules.py
Original file line number Diff line number Diff line change
Expand Up @@ -168,6 +168,15 @@ def __hash__(self):
],
)

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],
Expand All @@ -181,7 +190,7 @@ def __hash__(self):

catalyst = Module(
name="catalyst",
dependencies=[tags, core],
dependencies=[tags, sketch, core],
source_file_regexes=[
"sql/catalyst/",
],
Expand Down Expand Up @@ -295,15 +304,6 @@ def __hash__(self):
],
)

sketch = Module(
name="sketch",
dependencies=[tags],
source_file_regexes=[
"common/sketch/",
],
sbt_test_goals=["sketch/test"],
)

graphx = Module(
name="graphx",
dependencies=[tags, core],
Expand Down

0 comments on commit 441c33d

Please sign in to comment.