pylint 2

src/load_orchestrator.py

@@ -1,4 +1,7 @@
 #!.venv/bin/python
+"""
+Orchestratore dei worker che caricano i dati su dataraw
+"""
 
 # Import necessary libraries
 import logging
@@ -19,6 +22,7 @@ CSV_PROCESSING_DELAY = 0.2
 # Tempo di attesa se non ci sono record da elaborare
 NO_RECORD_SLEEP = 60
 
+
 async def worker(worker_id: int, cfg: object, pool: object) -> None:
     """Esegue il ciclo di lavoro per l'elaborazione dei file CSV.
 
@@ -39,7 +43,12 @@ async def worker(worker_id: int, cfg: object, pool: object) -> None:
         try:
             logger.info("Inizio elaborazione")
 
-            record = await get_next_csv_atomic(pool, cfg.dbrectable, WorkflowFlags.CSV_RECEIVED, WorkflowFlags.DATA_LOADED)
+            record = await get_next_csv_atomic(
+                pool,
+                cfg.dbrectable,
+                WorkflowFlags.CSV_RECEIVED,
+                WorkflowFlags.DATA_LOADED,
+            )
 
             if record:
                 success = await load_csv(record, cfg, pool)
@@ -50,8 +59,8 @@ async def worker(worker_id: int, cfg: object, pool: object) -> None:
                 logger.info("Nessun record disponibile")
                 await asyncio.sleep(NO_RECORD_SLEEP)
 
-        except Exception as e:
-            logger.error(f"Errore durante l'esecuzione: {e}", exc_info=1)
+        except Exception as e: # pylint: disable=broad-except
+            logger.error("Errore durante l'esecuzione: %s", e, exc_info=1)
             await asyncio.sleep(1)
 
 
@@ -59,7 +68,8 @@ async def load_csv(record: tuple, cfg: object, pool: object) -> bool:
     """Carica ed elabora un record CSV utilizzando il modulo di parsing appropriato.
 
     Args:
-        record: Una tupla contenente i dettagli del record CSV da elaborare (id, unit_type, tool_type, unit_name, tool_name).
+        record: Una tupla contenente i dettagli del record CSV da elaborare
+                (rec_id, unit_type, tool_type, unit_name, tool_name).
         cfg: L'oggetto di configurazione contenente i parametri del sistema.
         pool (object): Il pool di connessioni al database.
 
@@ -70,11 +80,16 @@ async def load_csv(record: tuple, cfg: object, pool: object) -> bool:
     debug_mode = logging.getLogger().getEffectiveLevel() == logging.DEBUG
     logger.debug("Inizio ricerca nuovo CSV da elaborare")
 
-    id, unit_type, tool_type, unit_name, tool_name = [
+    rec_id, unit_type, tool_type, unit_name, tool_name = [
         x.lower().replace(" ", "_") if isinstance(x, str) else x for x in record
     ]
     logger.info(
-        f"Trovato CSV da elaborare: ID={id}, Tipo={unit_type}_{tool_type}, Nome={unit_name}_{tool_name}"
+        "Trovato CSV da elaborare: ID=%s, Tipo=%s_%s, Nome=%s_%s",
+        rec_id,
+        unit_type,
+        tool_type,
+        unit_name,
+        tool_name,
     )
 
     # Costruisce il nome del modulo da caricare dinamicamente
@@ -87,27 +102,29 @@ async def load_csv(record: tuple, cfg: object, pool: object) -> bool:
     modulo = None
     for module_name in module_names:
         try:
-            logger.debug(f"Caricamento dinamico del modulo: {module_name}")
+            logger.debug("Caricamento dinamico del modulo: %s", module_name)
             modulo = importlib.import_module(module_name)
-            logger.info(f"Funzione 'main_loader' caricata dal modulo {module_name}")
+            logger.info("Funzione 'main_loader' caricata dal modulo %s", module_name)
             break
         except (ImportError, AttributeError) as e:
             logger.debug(
-                f"Modulo {module_name} non presente o non valido. {e}",
+                "Modulo %s non presente o non valido. %s",
+                module_name,
+                e,
                 exc_info=debug_mode,
             )
 
     if not modulo:
-        logger.error(f"Nessun modulo trovato {module_names}")
+        logger.error("Nessun modulo trovato %s", module_names)
         return False
 
     # Ottiene la funzione 'main_loader' dal modulo
     funzione = getattr(modulo, "main_loader")
 
     # Esegui la funzione
-    logger.info(f"Elaborazione con modulo {modulo} per ID={id}")
-    await funzione(cfg, id, pool)
-    logger.info(f"Elaborazione completata per ID={id}")
+    logger.info("Elaborazione con modulo %s per ID=%s", modulo, rec_id)
+    await funzione(cfg, rec_id, pool)
+    logger.info("Elaborazione completata per ID=%s", rec_id)
     return True