From 91a5227c25d50c05c422d32b03b1e81801984726 Mon Sep 17 00:00:00 2001
From: Mikko Siukola <mikko.siukola@cybercom.com>
Date: Thu, 3 Oct 2024 14:00:28 +0300
Subject: [PATCH] OK-591 Cleanup, use logger

---
 .../ovara/ajastus/SiirtotiedostoProsessi.java            | 4 ----
 .../impl/SiirtotiedostoProsessiRepositoryImpl.java       | 9 +++++++--
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/ovara-valintaperusteet/src/main/java/fi/vm/sade/service/valintaperusteet/ovara/ajastus/SiirtotiedostoProsessi.java b/ovara-valintaperusteet/src/main/java/fi/vm/sade/service/valintaperusteet/ovara/ajastus/SiirtotiedostoProsessi.java
index 96405a3e..80055a5f 100644
--- a/ovara-valintaperusteet/src/main/java/fi/vm/sade/service/valintaperusteet/ovara/ajastus/SiirtotiedostoProsessi.java
+++ b/ovara-valintaperusteet/src/main/java/fi/vm/sade/service/valintaperusteet/ovara/ajastus/SiirtotiedostoProsessi.java
@@ -56,10 +56,6 @@ public SiirtotiedostoProsessi(
   }
 
   public SiirtotiedostoProsessi(Object[] result) {
-    System.out.println("Parsitaan prosessi");
-    for (int i = 0; i < result.length; i++) {
-      System.out.println(result[i]);
-    }
     this.executionUuid = (String) result[0];
     this.windowStart = ((Timestamp) result[1]).toInstant().atOffset(ZoneOffset.UTC);
     this.windowEnd = ((Timestamp) result[2]).toInstant().atOffset(ZoneOffset.UTC);
diff --git a/ovara-valintaperusteet/src/main/java/fi/vm/sade/service/valintaperusteet/ovara/ajastus/impl/SiirtotiedostoProsessiRepositoryImpl.java b/ovara-valintaperusteet/src/main/java/fi/vm/sade/service/valintaperusteet/ovara/ajastus/impl/SiirtotiedostoProsessiRepositoryImpl.java
index 4610de77..ec20f9d4 100644
--- a/ovara-valintaperusteet/src/main/java/fi/vm/sade/service/valintaperusteet/ovara/ajastus/impl/SiirtotiedostoProsessiRepositoryImpl.java
+++ b/ovara-valintaperusteet/src/main/java/fi/vm/sade/service/valintaperusteet/ovara/ajastus/impl/SiirtotiedostoProsessiRepositoryImpl.java
@@ -1,12 +1,15 @@
 package fi.vm.sade.service.valintaperusteet.ovara.ajastus.impl;
 
 import fi.vm.sade.service.valintaperusteet.dao.AbstractJpaDAOImpl;
+import fi.vm.sade.service.valintaperusteet.ovara.ajastus.SiirtotiedostoAjastusService;
 import fi.vm.sade.service.valintaperusteet.ovara.ajastus.SiirtotiedostoProsessi;
 import fi.vm.sade.service.valintaperusteet.ovara.ajastus.SiirtotiedostoProsessiRepository;
 import javax.persistence.EntityManager;
 import javax.persistence.PersistenceContext;
 import javax.persistence.Query;
 import javax.transaction.Transactional;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import org.springframework.jdbc.core.JdbcTemplate;
 import org.springframework.stereotype.Repository;
 
@@ -15,8 +18,10 @@
 public class SiirtotiedostoProsessiRepositoryImpl
     extends AbstractJpaDAOImpl<SiirtotiedostoProsessi, Long>
     implements SiirtotiedostoProsessiRepository {
-  @PersistenceContext private EntityManager entityManager;
 
+  private static final Logger logger = LoggerFactory.getLogger(SiirtotiedostoAjastusService.class);
+
+  @PersistenceContext private EntityManager entityManager;
   private final JdbcTemplate jdbcTemplate;
 
   public SiirtotiedostoProsessiRepositoryImpl(JdbcTemplate jdbcTemplate) {
@@ -35,7 +40,7 @@ public SiirtotiedostoProsessi findLatestSuccessful() {
 
   @Override
   public void persist(SiirtotiedostoProsessi sp) {
-    System.out.println("Persisting: " + sp);
+    logger.info("Persisting: {}", sp);
     String infoStr = sp.getInfo() != null ? sp.getInfo() : "{}";
     this.jdbcTemplate.update(
         "insert into siirtotiedosto (execution_uuid, window_start, window_end, run_start, run_end, info, success, error_message) "