Skip to content

Commit

Permalink
[SPARK-50736][INFRA] Enable testing module pyspark-logger
Browse files Browse the repository at this point in the history
### What changes were proposed in this pull request?
Enable testing module `pyspark-logger`

### Why are the changes needed?
this testing module is missing in CI, so following tests were always skipped

https://github.com/apache/spark/blob/85d92d7c3a6a38b1b6cfc667caac9176fab5813b/dev/sparktestsupport/modules.py#L1441-L1450

### Does this PR introduce _any_ user-facing change?
no, infra-only

### How was this patch tested?
ci

### Was this patch authored or co-authored using generative AI tooling?
no

Closes #49367 from zhengruifeng/py_test_logging.

Authored-by: Ruifeng Zheng <ruifengz@apache.org>
Signed-off-by: Ruifeng Zheng <ruifengz@apache.org>
  • Loading branch information
zhengruifeng committed Jan 6, 2025
1 parent 0d3f4cd commit 47825e5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build_and_test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -499,7 +499,7 @@ jobs:
- >-
pyspark-sql, pyspark-resource, pyspark-testing
- >-
pyspark-core, pyspark-errors, pyspark-streaming
pyspark-core, pyspark-errors, pyspark-streaming, pyspark-logger
- >-
pyspark-mllib, pyspark-ml, pyspark-ml-connect
- >-
Expand Down
2 changes: 1 addition & 1 deletion dev/sparktestsupport/modules.py
Original file line number Diff line number Diff line change
Expand Up @@ -1438,7 +1438,7 @@ def __hash__(self):
],
)

pyspark_logging = Module(
pyspark_logger = Module(
name="pyspark-logger",
dependencies=[],
source_file_regexes=["python/pyspark/logger"],
Expand Down

0 comments on commit 47825e5

Please sign in to comment.