diff --git a/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/snapshot/SnapshotDiffManager.java b/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/snapshot/SnapshotDiffManager.java index 1e0b5f7d8771..51638ac9c722 100644 --- a/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/snapshot/SnapshotDiffManager.java +++ b/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/snapshot/SnapshotDiffManager.java @@ -730,10 +730,10 @@ private synchronized SnapshotDiffResponse submitSnapDiffJob( // If executor cannot take any more job, remove the job form DB and return // the Rejected Job status with wait time. try { + updateJobStatus(jobKey, QUEUED, IN_PROGRESS); snapDiffExecutor.execute(() -> generateSnapshotDiffReport(jobKey, jobId, volumeName, bucketName, fromSnapshotName, toSnapshotName, forceFullDiff, disableNativeDiff)); - updateJobStatus(jobKey, QUEUED, IN_PROGRESS); return new SnapshotDiffResponse( new SnapshotDiffReportOzone(snapshotRoot.toString(), volumeName, bucketName, fromSnapshotName, toSnapshotName, new ArrayList<>(), diff --git a/hadoop-ozone/ozone-manager/src/test/java/org/apache/hadoop/ozone/om/snapshot/TestSnapshotDiffManager.java b/hadoop-ozone/ozone-manager/src/test/java/org/apache/hadoop/ozone/om/snapshot/TestSnapshotDiffManager.java index 2ffbaa44d828..0ea625a0e064 100644 --- a/hadoop-ozone/ozone-manager/src/test/java/org/apache/hadoop/ozone/om/snapshot/TestSnapshotDiffManager.java +++ b/hadoop-ozone/ozone-manager/src/test/java/org/apache/hadoop/ozone/om/snapshot/TestSnapshotDiffManager.java @@ -151,7 +151,6 @@ import org.apache.ozone.rocksdiff.DifferSnapshotInfo; import org.apache.ozone.rocksdiff.RocksDBCheckpointDiffer; import org.apache.ozone.rocksdiff.RocksDiffUtils; -import org.apache.ozone.test.tag.Flaky; import org.apache.ratis.util.ExitUtils; import org.apache.ratis.util.TimeDuration; import org.apache.ratis.util.function.UncheckedAutoCloseableSupplier; @@ -1304,7 +1303,6 @@ public void testGenerateDiffReportFailure() throws IOException { * startup. */ @Test - @Flaky("HDDS-10490") public void testLoadJobsOnStartUp() throws Exception { for (int i = 0; i < snapshotInfoList.size(); i++) { uploadSnapshotDiffJobToDb(snapshotInfo, snapshotInfoList.get(i), @@ -1618,7 +1616,6 @@ public void testGetSnapshotDiffReportHappyCase() throws Exception { * short-circuited based on previous one. */ @Test - @Flaky("HDDS-12361") public void testGetSnapshotDiffReportJob() throws Exception { for (int i = 0; i < jobStatuses.size(); i++) { uploadSnapshotDiffJobToDb(snapshotInfo, snapshotInfoList.get(i),