Skip to content

Commit a50f453

Browse files
committed
[SPARK-52624] Upgrade spotbugs-tool to 4.9.3 and spotbugs-plugin to 6.1.13
### What changes were proposed in this pull request? This PR aims to upgrade - `spotbugs-tool` from `4.8.6` to `4.9.3` - `spotbugs-plugin` from `6.0.17` to 6.1.13 While updating the above, the following are fixed. ``` > Task :spark-operator:spotbugsTest FAILED M D US: Suppressing annotation on the class org.apache.spark.k8s.operator.probe.HealthProbeTest is unnecessary At HealthProbeTest.java:[lines 52-192] M D US: Suppressing annotation on the class org.apache.spark.k8s.operator.metrics.source.KubernetesMetricsInterceptorTest is unnecessary At KubernetesMetricsInterceptorTest.java:[lines 55-153] ``` ### Why are the changes needed? To bring the latest updates. ### Does this PR introduce _any_ user-facing change? No behavior change. ### How was this patch tested? Pass the CIs. ### Was this patch authored or co-authored using generative AI tooling? No. Closes apache#265 from dongjoon-hyun/SPARK-52624. Authored-by: Dongjoon Hyun <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
1 parent 5885215 commit a50f453

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

gradle/libs.versions.toml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,8 @@ mockito = "5.18.0"
3131
# Build Analysis
3232
checkstyle = "10.23.1"
3333
pmd = "7.13.0"
34-
spotbugs-tool = "4.8.6"
35-
spotbugs-plugin = "6.0.17"
34+
spotbugs-tool = "4.9.3"
35+
spotbugs-plugin = "6.1.13"
3636
spotless-plugin = "6.25.0"
3737

3838
# Packaging

spark-operator/src/test/java/org/apache/spark/k8s/operator/metrics/source/KubernetesMetricsInterceptorTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@
5050
@EnableKubernetesMockClient(crud = true)
5151
@TestMethodOrder(MethodOrderer.OrderAnnotation.class)
5252
@SuppressFBWarnings(
53-
value = {"UWF_UNWRITTEN_FIELD", "NP_UNWRITTEN_FIELD", "UUF_UNUSED_FIELD"},
53+
value = {"UWF_UNWRITTEN_FIELD", "NP_UNWRITTEN_FIELD"},
5454
justification = "Unwritten fields are covered by Kubernetes mock client")
5555
class KubernetesMetricsInterceptorTest {
5656

spark-operator/src/test/java/org/apache/spark/k8s/operator/probe/HealthProbeTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@
4747

4848
@EnableKubernetesMockClient(crud = true)
4949
@SuppressFBWarnings(
50-
value = {"UWF_UNWRITTEN_FIELD", "NP_UNWRITTEN_FIELD", "UUF_UNUSED_FIELD"},
50+
value = {"UWF_UNWRITTEN_FIELD"},
5151
justification = "Unwritten fields are covered by Kubernetes mock client")
5252
class HealthProbeTest {
5353
private static Operator operator;

0 commit comments

Comments
 (0)