From 47825e502a22ff8c14a27f05a09b953bf403ae65 Mon Sep 17 00:00:00 2001 From: Ruifeng Zheng Date: Mon, 6 Jan 2025 21:33:13 +0800 Subject: [PATCH] [SPARK-50736][INFRA] Enable testing module `pyspark-logger` ### 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 Signed-off-by: Ruifeng Zheng --- .github/workflows/build_and_test.yml | 2 +- dev/sparktestsupport/modules.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/build_and_test.yml b/.github/workflows/build_and_test.yml index a46b90b4913ee..614fdd49d83df 100644 --- a/.github/workflows/build_and_test.yml +++ b/.github/workflows/build_and_test.yml @@ -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 - >- diff --git a/dev/sparktestsupport/modules.py b/dev/sparktestsupport/modules.py index 2a19439bd92f9..71817a64b3581 100644 --- a/dev/sparktestsupport/modules.py +++ b/dev/sparktestsupport/modules.py @@ -1438,7 +1438,7 @@ def __hash__(self): ], ) -pyspark_logging = Module( +pyspark_logger = Module( name="pyspark-logger", dependencies=[], source_file_regexes=["python/pyspark/logger"],