From 37d64389b9658f9c0a6367a1c3b3abffb430d0e5 Mon Sep 17 00:00:00 2001
From: Calvin Kirs <kirs@apache.org>
Date: Fri, 16 Feb 2024 14:34:19 +0800
Subject: [PATCH] [Improve](Job)Return directly when the job status does not
 need to change (#31028)

---
 .../main/java/org/apache/doris/job/base/AbstractJob.java    | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/fe/fe-core/src/main/java/org/apache/doris/job/base/AbstractJob.java b/fe/fe-core/src/main/java/org/apache/doris/job/base/AbstractJob.java
index 091ac158c1..78d7598bf1 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/job/base/AbstractJob.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/job/base/AbstractJob.java
@@ -244,11 +244,11 @@ public abstract class AbstractJob<T extends AbstractTask, C> implements Job<T, C
         if (null == newJobStatus) {
             throw new IllegalArgumentException("jobStatus cannot be null");
         }
+        if (jobStatus == newJobStatus) {
+            return;
+        }
         String errorMsg = String.format("Can't update job %s status to the %s status",
                 jobStatus.name(), newJobStatus.name());
-        if (jobStatus == newJobStatus) {
-            throw new IllegalArgumentException(errorMsg);
-        }
         if (newJobStatus.equals(JobStatus.RUNNING) && !jobStatus.equals(JobStatus.PAUSED)) {
             throw new IllegalArgumentException(errorMsg);
         }