[Enhancement] move the updateExecStatus out of Coordinator Lock (#49073)

Signed-off-by: Murphy <[email protected]>
StarRocks · Jul 30, 2024 · 07b9ee1 · 07b9ee1
1 parent df5a254
commit 07b9ee1
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/fe/fe-core/src/main/java/com/starrocks/qe/DefaultCoordinator.java b/fe/fe-core/src/main/java/com/starrocks/qe/DefaultCoordinator.java
@@ -956,12 +956,12 @@ public void updateFragmentExecStatus(TReportExecStatusParams params) {
 
         queryProfile.updateProfile(execState, params);
 
+        if (!execState.updateExecStatus(params)) {
+            return;
+        }
+
         lock();
         try {
-            if (!execState.updateExecStatus(params)) {
-                return;
-            }
-
             queryProfile.updateLoadChannelProfile(params);
         } finally {
             unlock();