You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by do...@apache.org on 2025/05/14 17:29:16 UTC
(spark-kubernetes-operator) branch main updated: [SPARK-52135] Remove `powermock-core` dependency
This is an automated email from the ASF dual-hosted git repository.
dongjoon pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/spark-kubernetes-operator.git
The following commit(s) were added to refs/heads/main by this push:
new a28b1b7 [SPARK-52135] Remove `powermock-core` dependency
a28b1b7 is described below
commit a28b1b70562c1ffe2d5a7a22371678add6e78585
Author: Dongjoon Hyun <do...@apache.org>
AuthorDate: Wed May 14 10:29:10 2025 -0700
[SPARK-52135] Remove `powermock-core` dependency
### What changes were proposed in this pull request?
This PR aims to remove `powermock-core` dependency.
### Why are the changes needed?
We had better minimize the dependency list by reducing old dependency like `powermock` which was released on 2020-11-01.
- https://github.com/powermock/powermock/releases/tag/powermock-2.0.9
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Pass the CIs.
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #198 from dongjoon-hyun/SPARK-52135.
Authored-by: Dongjoon Hyun <do...@apache.org>
Signed-off-by: Dongjoon Hyun <do...@apache.org>
---
gradle/libs.versions.toml | 2 --
spark-operator/build.gradle | 1 -
.../test/java/org/apache/spark/k8s/operator/utils/TestUtils.java | 8 ++++++--
3 files changed, 6 insertions(+), 5 deletions(-)
diff --git a/gradle/libs.versions.toml b/gradle/libs.versions.toml
index b08523e..14d72af 100644
--- a/gradle/libs.versions.toml
+++ b/gradle/libs.versions.toml
@@ -27,7 +27,6 @@ log4j = "2.24.3"
junit = "5.12.2"
jacoco = "0.8.13"
mockito = "5.17.0"
-powermock = "2.0.9"
# Build Analysis
checkstyle = "10.17.0"
@@ -64,7 +63,6 @@ mockito-core = { group = "org.mockito", name = "mockito-core", version.ref = "mo
junit-bom = { group = "org.junit", name = "junit-bom", version.ref = "junit"}
junit-jupiter = { group = "org.junit.jupiter", name = "junit-jupiter", version.ref = "junit"}
junit-platform-launcher = { group = "org.junit.platform", name = "junit-platform-launcher"}
-powermock-core = { group = "org.powermock", name = "powermock-core", version.ref = "powermock"}
spotbugs-gradle-plugin = { group = "com.github.spotbugs.snom", name = "spotbugs-gradle-plugin", version.ref = "spotbugs-plugin" }
spotless-plugin-gradle = { group = "com.diffplug.spotless", name = "spotless-plugin-gradle", version.ref = "spotless-plugin" }
shadow = { group = "com.gradleup.shadow", name = "shadow-gradle-plugin", version.ref = "shadow-jar-plugin"}
diff --git a/spark-operator/build.gradle b/spark-operator/build.gradle
index 1d41733..5d3a690 100644
--- a/spark-operator/build.gradle
+++ b/spark-operator/build.gradle
@@ -77,7 +77,6 @@ dependencies {
testImplementation(libs.mockwebserver)
testImplementation platform(libs.junit.bom)
testImplementation(libs.junit.jupiter)
- testImplementation(libs.powermock.core)
testRuntimeOnly(libs.junit.platform.launcher)
testImplementation(libs.mockito.core)
diff --git a/spark-operator/src/test/java/org/apache/spark/k8s/operator/utils/TestUtils.java b/spark-operator/src/test/java/org/apache/spark/k8s/operator/utils/TestUtils.java
index 50bd2ff..be06d09 100644
--- a/spark-operator/src/test/java/org/apache/spark/k8s/operator/utils/TestUtils.java
+++ b/spark-operator/src/test/java/org/apache/spark/k8s/operator/utils/TestUtils.java
@@ -26,7 +26,7 @@ import java.io.File;
import java.util.Map;
import io.fabric8.kubernetes.api.model.ObjectMeta;
-import org.powermock.reflect.Whitebox;
+import org.apache.commons.lang3.reflect.FieldUtils;
import org.apache.spark.k8s.operator.Constants;
import org.apache.spark.k8s.operator.SparkApplication;
@@ -67,6 +67,10 @@ public final class TestUtils {
}
public static <T> void setConfigKey(ConfigOption<T> configKey, T newValue) {
- Whitebox.setInternalState(configKey, "defaultValue", newValue);
+ try {
+ FieldUtils.writeField(configKey, "defaultValue", newValue, true);
+ } catch (IllegalAccessException e) {
+ throw new UnsupportedOperationException(e);
+ }
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org