From a0ab619aebcb86e38375a8556fd88c0c3255cf03 Mon Sep 17 00:00:00 2001
From: Cheng Zhikai <102442786+zxc11086@users.noreply.github.com>
Date: Tue, 19 Aug 2025 09:42:15 +0800
Subject: [PATCH] Fix: ensure update_progress loop always waits between
 iterations (#9528)

Move stop_event.wait(6) into finally block so that even when an
exception occurs, the loop still sleeps before retrying. This prevents
busy looping and excessive error logs when Redis connection fails.

### Type of change

- [x] Bug Fix (non-breaking change which fixes an issue)
---
 api/ragflow_server.py | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/api/ragflow_server.py b/api/ragflow_server.py
index 8d0432012..0dbeb771b 100644
--- a/api/ragflow_server.py
+++ b/api/ragflow_server.py
@@ -59,11 +59,14 @@ def update_progress():
             if redis_lock.acquire():
                 DocumentService.update_progress()
                 redis_lock.release()
-            stop_event.wait(6)
         except Exception:
             logging.exception("update_progress exception")
         finally:
-            redis_lock.release()
+            try:
+                redis_lock.release()
+            except Exception:
+                logging.exception("update_progress exception")
+            stop_event.wait(6)
 
 def signal_handler(sig, frame):
     logging.info("Received interrupt signal, shutting down...")