MultiBackendJobManager: fix another SettingWithCopyWarning related bug

seems like pandas is handy to build foot guns maybe related to #641
Open-EO · Oct 11, 2024 · 5b1e8fa · 5b1e8fa
1 parent 632e239
commit 5b1e8fa
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/openeo/extra/job_management.py b/openeo/extra/job_management.py
@@ -505,12 +505,11 @@ def _job_update_loop(
                 backend_load = per_backend.get(backend_name, 0)
                 if backend_load < self.backends[backend_name].parallel_jobs:
                     to_add = self.backends[backend_name].parallel_jobs - backend_load
-                    to_launch = not_started.iloc[0:to_add]
-                    for i in to_launch.index:
+                    for i in not_started.index[0:to_add]:
                         self._launch_job(start_job, df=not_started, i=i, backend_name=backend_name, stats=stats)
                         stats["job launch"] += 1
 
-                        job_db.persist(to_launch)
+                        job_db.persist(not_started.loc[i : i + 1])
                         stats["job_db persist"] += 1
 
     def _launch_job(self, start_job, df, i, backend_name, stats: Optional[dict] = None):