[SPARK-52137] Upgrade pmd to 7.13.0

Author: dongjoon-hyun

build-tools/docs-utils/src/main/java/org/apache/spark/k8s/operator/utils/DocTable.java

@@ -57,8 +57,7 @@ public void flush(PrintWriter writer) {
   private String joinRow(List<String> elements) {
     StringBuilder stringBuilder = new StringBuilder(ROW_SEPARATOR);
     for (String element : elements) {
-      stringBuilder.append(element);
-      stringBuilder.append(ROW_SEPARATOR);
+      stringBuilder.append(element).append(ROW_SEPARATOR);
     }
     if (elements.size() < columns) {
       // Append empty cells to end if needed

config/pmd/ruleset.xml

@@ -22,8 +22,13 @@
     Apache Spark Kubernetes Operator Ruleset
   </description>
   <rule ref="category/java/bestpractices.xml">
-    <exclude name="JUnitAssertionsShouldIncludeMessage" />
-    <exclude name="JUnitTestContainsTooManyAsserts" />
+    <exclude name="GuardLogStatement" />
+    <exclude name="ImplicitFunctionalInterface" />
+    <exclude name="UnitTestAssertionsShouldIncludeMessage" />
+    <exclude name="UnitTestContainsTooManyAsserts" />
+    <exclude name="UnitTestShouldIncludeAssert" />
+    <exclude name="UnusedLocalVariable" />
+    <exclude name="LooseCoupling" />
   </rule>
   <rule ref="category/java/codestyle.xml">
     <exclude name="AtLeastOneConstructor" />
@@ -36,16 +41,24 @@
     <exclude name="OnlyOneReturn" />
     <exclude name="ShortVariable" />
     <exclude name="TooManyStaticImports" />
+    <exclude name="UnnecessaryCast" />
+    <exclude name="UnnecessaryFullyQualifiedName" />
+    <exclude name="UseExplicitTypes" />
     <exclude name="UseUnderscoresInNumericLiterals" />
   </rule>
   <rule ref="category/java/design.xml">
     <exclude name="AbstractClassWithoutAnyMethod" />
     <exclude name="AvoidCatchingGenericException" />
+    <exclude name="ClassWithOnlyPrivateConstructorsShouldBeFinal" />
     <exclude name="CognitiveComplexity" />
+    <exclude name="CouplingBetweenObjects" />
     <exclude name="CyclomaticComplexity" />
+    <exclude name="DataClass" />
     <exclude name="ExcessiveImports" />
+    <exclude name="FinalFieldCouldBeStatic" />
     <exclude name="LawOfDemeter" />
     <exclude name="NPathComplexity" />
+    <exclude name="SimplifyBooleanReturns" />
     <exclude name="TooManyMethods" />
   </rule>
   <rule ref="category/java/documentation.xml">
@@ -67,13 +80,15 @@
     <exclude name="UseProperClassLoader" />
   </rule>
   <rule ref="category/java/multithreading.xml">
+    <exclude name="AvoidSynchronizedStatement" />
     <exclude name="DoNotUseThreads" />
     <exclude name="UseConcurrentHashMap" />
   </rule>
   <rule ref="category/java/performance.xml">
     <exclude name="AvoidFileStream" />
     <exclude name="AvoidInstantiatingObjectsInLoops" />
     <exclude name="RedundantFieldInitializer" />
+    <exclude name="UseArraysAsList" />
   </rule>
   <rule ref="category/java/security.xml"/>
   <!-- exclude on generated files -->

gradle/libs.versions.toml

@@ -30,7 +30,7 @@ mockito = "5.17.0"
 
 # Build Analysis
 checkstyle = "10.23.1"
-pmd = "6.55.0"
+pmd = "7.13.0"
 spotbugs-tool = "4.8.6"
 spotbugs-plugin = "6.0.17"
 spotless-plugin = "6.25.0"