[8.2] Ensure accurate backgroundIndexing status in debugInfo (#812)

Ensure accurate `backgroundIndexing` status in debugInfo (#811)

convert updateJobMutex to mutable

check backgroundIndexing under the lock

(cherry picked from commit 46e69b4)

Co-authored-by: meiravgri <[email protected]>

Author: github-actions[bot]

src/VecSim/algorithms/svs/svs_tiered.h

@@ -215,7 +215,7 @@ class TieredSVSIndex : public VecSimTieredIndex<DataType, float> {
     std::atomic_flag indexUpdateScheduled = ATOMIC_FLAG_INIT;
     // Used to prevent running multiple index update jobs in parallel.
     // Even if update jobs scheduled sequentially, they can be started in parallel.
-    std::mutex updateJobMutex;
+    mutable std::mutex updateJobMutex;
 
     // The reason of following container just to properly destroy jobs which not executed yet
     SVSMultiThreadJob::JobsRegistry uncompletedJobs;
@@ -780,6 +780,8 @@ class TieredSVSIndex : public VecSimTieredIndex<DataType, float> {
                                        .indexUpdateScheduled =
                                            static_cast<bool>(this->indexUpdateScheduled.test())};
         info.tieredInfo.specificTieredBackendInfo.svsTieredInfo = svsTieredInfo;
+        // prevent parallel updates
+        std::lock_guard<std::mutex> lock(this->updateJobMutex);
         info.tieredInfo.backgroundIndexing =
             svsTieredInfo.indexUpdateScheduled && info.tieredInfo.frontendCommonInfo.indexSize > 0
                 ? VecSimBool_TRUE