From 50d801cf261b16df88de54f712be5d45a1a98495 Mon Sep 17 00:00:00 2001 From: Dongjoon Hyun Date: Thu, 7 May 2026 10:55:31 -0700 Subject: [PATCH] [SPARK-56780] Upgrade PMD to 7.24.0 This PR aims to upgrade PMD from 7.23.0 to 7.24.0. To stay current with the latest stable PMD release. - https://github.com/pmd/pmd/releases/tag/pmd_releases%2F7.24.0 PMD 7.24.0 introduces a new `UnnecessaryWarningSuppression` rule that flagged one redundant `@SuppressWarnings("PMD.UnitTestShouldIncludeAssert")` in `SparkOperatorConfigMapReconcilerTest.sanityTest`. The annotation is removed because the lambda inside `await().untilAsserted(...)` already contains an `assertThat(...)` call, so the suppression is no longer needed. No. Pass the CIs. Generated-by: Claude Code Co-Authored-By: Claude Opus 4.7 (1M context) --- gradle/libs.versions.toml | 2 +- .../operator/config/SparkOperatorConfigMapReconcilerTest.java | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/gradle/libs.versions.toml b/gradle/libs.versions.toml index b1ec497e..10ba6c7e 100644 --- a/gradle/libs.versions.toml +++ b/gradle/libs.versions.toml @@ -32,7 +32,7 @@ mockito = "5.23.0" # Build Analysis checkstyle = "13.4.2" -pmd = "7.23.0" +pmd = "7.24.0" spotbugs-tool = "4.9.8" spotbugs-plugin = "6.5.4" spotless-plugin = "8.4.0" diff --git a/spark-operator/src/test/java/org/apache/spark/k8s/operator/config/SparkOperatorConfigMapReconcilerTest.java b/spark-operator/src/test/java/org/apache/spark/k8s/operator/config/SparkOperatorConfigMapReconcilerTest.java index 9ef941ee..d8f81e79 100644 --- a/spark-operator/src/test/java/org/apache/spark/k8s/operator/config/SparkOperatorConfigMapReconcilerTest.java +++ b/spark-operator/src/test/java/org/apache/spark/k8s/operator/config/SparkOperatorConfigMapReconcilerTest.java @@ -65,7 +65,6 @@ void stopController() { } @Test - @SuppressWarnings("PMD.UnitTestShouldIncludeAssert") void sanityTest() { client.resource(testConfigMap()).create();