IGNITE-28390 Better fix for compute job spotbugs

modules/compute/jobs.gradle

@@ -53,7 +53,6 @@ dependencies {
     jobsImplementation project(':ignite-transactions')
     jobsImplementation project(':ignite-runner')
     jobsImplementation testFixtures(project(':ignite-runner'))
-    jobsCompileOnly libs.spotbugs.annotations
     unit1Implementation project(':ignite-api')
     unit2Implementation project(':ignite-api')
 }

modules/compute/src/integrationTest/java/org/apache/ignite/internal/compute/ItComputeBaseTest.java

@@ -906,11 +906,11 @@ public void observableTsIsPropagatedToTargetNode() {
         ObservableTimestampResult jobRes = execution.resultAsync().join();
 
         // The per-job tracker should have the client's observable timestamp.
-        HybridTimestamp jobObservableTs = HybridTimestamp.nullableHybridTimestamp(jobRes.perJobTimestamp);
+        HybridTimestamp jobObservableTs = HybridTimestamp.nullableHybridTimestamp(jobRes.perJobTimestamp());
         assertThat(jobObservableTs, is(localObservableTs));
 
         // The node's global tracker should NOT be updated by the compute job.
-        HybridTimestamp targetNodeTsAfter = HybridTimestamp.nullableHybridTimestamp(jobRes.nodeGlobalTimestamp);
+        HybridTimestamp targetNodeTsAfter = HybridTimestamp.nullableHybridTimestamp(jobRes.nodeGlobalTimestamp());
         assertThat(targetNodeTsAfter, is(targetNodeTsBefore));
         assertThat(targetNodeTsAfter, not(jobObservableTs));
     }

modules/compute/src/integrationTest/java/org/example/jobs/embedded/ObservableTimestampResult.java

@@ -21,8 +21,8 @@
  * Result of {@link ObservableTimestampJob} containing both per-job and node global observable timestamps.
  */
 public class ObservableTimestampResult {
-    public long perJobTimestamp;
-    public long nodeGlobalTimestamp;
+    private long perJobTimestamp;
+    private long nodeGlobalTimestamp;
 
     public ObservableTimestampResult() {
     }
@@ -31,4 +31,12 @@ public ObservableTimestampResult(long perJobTimestamp, long nodeGlobalTimestamp)
         this.perJobTimestamp = perJobTimestamp;
         this.nodeGlobalTimestamp = nodeGlobalTimestamp;
     }
+
+    public long perJobTimestamp() {
+        return perJobTimestamp;
+    }
+
+    public long nodeGlobalTimestamp() {
+        return nodeGlobalTimestamp;
+    }
 }

modules/compute/src/jobs/java/org/example/jobs/standalone/ObservableTimestampResult.java

@@ -17,18 +17,12 @@
 
 package org.example.jobs.standalone;
 
-import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
-
 /**
  * Result of {@link ObservableTimestampJob} containing both per-job and node global observable timestamps.
  */
-@SuppressFBWarnings(
-        value = "URF_UNREAD_PUBLIC_OR_PROTECTED_FIELD",
-        justification = "Fields are read in integration tests via deserialized object"
-)
 public class ObservableTimestampResult {
-    public long perJobTimestamp;
-    public long nodeGlobalTimestamp;
+    private long perJobTimestamp;
+    private long nodeGlobalTimestamp;
 
     public ObservableTimestampResult() {
     }
@@ -37,4 +31,12 @@ public ObservableTimestampResult(long perJobTimestamp, long nodeGlobalTimestamp)
         this.perJobTimestamp = perJobTimestamp;
         this.nodeGlobalTimestamp = nodeGlobalTimestamp;
     }
+
+    public long perJobTimestamp() {
+        return perJobTimestamp;
+    }
+
+    public long nodeGlobalTimestamp() {
+        return nodeGlobalTimestamp;
+    }
 }