Merge pull request #41 from abes-esr/CDE-277-Mesurer-et-noter-les-per…

…formances-sur-plusieurs-fichiers

Ajout @ExecutionTime sur méthodes du projet faisant appel à la basexml

kbart-webservices/src/main/java/fr/abes/convergence/kbartws/component/BaseXmlFunctionsCaller.java

@@ -1,5 +1,6 @@
 package fr.abes.convergence.kbartws.component;
 
+import fr.abes.convergence.kbartws.utils.ExecutionTime;
 import org.hibernate.annotations.ColumnTransformer;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.data.jpa.repository.query.EscapeCharacter;
@@ -14,6 +15,7 @@ public class BaseXmlFunctionsCaller {
     @Autowired
     private JdbcTemplate baseXmlJdbcTemplate;
 
+    @ExecutionTime
     @ColumnTransformer(read = "XMLSERIALIZE (CONTENT data_xml as CLOB)", write = "NULLSAFE_XMLTYPE(?)")
     public String issnToPpn(String issn) throws SQLRecoverableException, UncategorizedSQLException {
         StringBuilder request = new StringBuilder("SELECT AUTORITES.ISSN2PPNJSON('");
@@ -22,6 +24,7 @@ public String issnToPpn(String issn) throws SQLRecoverableException, Uncategoriz
         return baseXmlJdbcTemplate.queryForObject(request.toString(), String.class);
     }
 
+    @ExecutionTime
     @ColumnTransformer(read = "XMLSERIALIZE (CONTENT data_xml as CLOB)", write = "NULLSAFE_XMLTYPE(?)")
     public String isbnToPpn(String isbn) throws SQLRecoverableException, UncategorizedSQLException {
         StringBuilder request = new StringBuilder("SELECT AUTORITES.ISBN2PPNJSON('");
@@ -30,6 +33,7 @@ public String isbnToPpn(String isbn) throws SQLRecoverableException, Uncategoriz
         return baseXmlJdbcTemplate.queryForObject(request.toString(), String.class);
     }
 
+    @ExecutionTime
     @ColumnTransformer(read = "XMLSERIALIZE (CONTENT data_xml as CLOB)", write = "NULLSAFE_XMLTYPE(?)")
     public String baconProvider035(Integer provider) throws SQLRecoverableException, UncategorizedSQLException {
         StringBuilder request = new StringBuilder("SELECT AUTORITES.BACON_PROVIDER_035_JSON(");
@@ -38,6 +42,7 @@ public String baconProvider035(Integer provider) throws SQLRecoverableException,
         return baseXmlJdbcTemplate.queryForObject(request.toString(), String.class);
     }
 
+    @ExecutionTime
     @ColumnTransformer(read = "XMLSERIALIZE (CONTENT data_xml as CLOB)", write = "NULLSAFE_XMLTYPE(?)")
     public String doiToPpn(String doi) throws SQLRecoverableException, UncategorizedSQLException {
         StringBuilder request = new StringBuilder("select XMLTRANSFORM(XMLROOT(XMLElement(\"sudoc\",AUTORITES.DOI2PNN('");

kbart-webservices/src/main/java/fr/abes/convergence/kbartws/repository/BiblioTableFrbr4XXRepository.java

@@ -1,10 +1,12 @@
 package fr.abes.convergence.kbartws.repository;
 
 import fr.abes.convergence.kbartws.entity.BiblioTableFrbr4XX;
+import fr.abes.convergence.kbartws.utils.ExecutionTime;
 import org.springframework.data.jpa.repository.JpaRepository;
 
 import java.util.List;
 
 public interface BiblioTableFrbr4XXRepository extends JpaRepository<BiblioTableFrbr4XX, Integer> {
+    @ExecutionTime
     List<BiblioTableFrbr4XX> findAllByTagAndDatas(String tag, String datas);
 }

kbart-webservices/src/main/java/fr/abes/convergence/kbartws/repository/NoticesBibioRepository.java

@@ -1,12 +1,14 @@
 package fr.abes.convergence.kbartws.repository;
 
 import fr.abes.convergence.kbartws.entity.NoticesBibio;
+import fr.abes.convergence.kbartws.utils.ExecutionTime;
 import org.springframework.data.jpa.repository.JpaRepository;
 import org.springframework.stereotype.Repository;
 
 import java.util.Optional;
 
 @Repository
 public interface NoticesBibioRepository extends JpaRepository<NoticesBibio, Integer> {
+    @ExecutionTime
     Optional<NoticesBibio> findByPpn(String ppn);
 }

kbart-webservices/src/main/java/fr/abes/convergence/kbartws/service/NoticeService.java

@@ -7,6 +7,7 @@
 import fr.abes.convergence.kbartws.exception.IllegalPpnException;
 import fr.abes.convergence.kbartws.repository.BiblioTableFrbr4XXRepository;
 import fr.abes.convergence.kbartws.repository.NoticesBibioRepository;
+import fr.abes.convergence.kbartws.utils.ExecutionTime;
 import lombok.extern.slf4j.Slf4j;
 import org.springframework.stereotype.Service;
 
@@ -17,6 +18,7 @@
 import java.util.Optional;
 
 @Service
+@Slf4j
 public class NoticeService {
     private final NoticesBibioRepository noticesBibioRepository;
 
@@ -44,7 +46,9 @@ public NoticeXml getNoticeByPpn(String ppn) throws IllegalPpnException, IOExcept
         return null;
     }
 
+    @ExecutionTime
     public List<String> getEquivalentElectronique(NoticeXml notice) throws IOException, IllegalPpnException {
+        log.debug("entrée dans getEquivalentElectronique");
         List<String> ppnlies;
 
         //on cherche une 452$0 dans la notice

kbart-webservices/src/main/java/fr/abes/convergence/kbartws/utils/ExecutionTime.java

@@ -0,0 +1,11 @@
+package fr.abes.convergence.kbartws.utils;
+
+import java.lang.annotation.ElementType;
+import java.lang.annotation.Retention;
+import java.lang.annotation.RetentionPolicy;
+import java.lang.annotation.Target;
+
+@Retention(RetentionPolicy.RUNTIME)
+@Target(ElementType.METHOD)
+public @interface ExecutionTime {
+}

kbart-webservices/src/main/java/fr/abes/convergence/kbartws/utils/ExecutionTimeAspect.java

@@ -0,0 +1,24 @@
+package fr.abes.convergence.kbartws.utils;
+
+import lombok.extern.slf4j.Slf4j;
+import org.aspectj.lang.ProceedingJoinPoint;
+import org.aspectj.lang.annotation.Around;
+import org.aspectj.lang.annotation.Aspect;
+
+@Aspect
+@Slf4j
+public class ExecutionTimeAspect {
+
+    @Around("@annotation(ExecutionTime)")
+    public Object measureExecutionTime(ProceedingJoinPoint joinPoint) throws Throwable {
+        long startTime = System.currentTimeMillis();
+
+        Object result = joinPoint.proceed();
+
+        long endTime = System.currentTimeMillis();
+        double executionTime = (endTime - startTime) / 1000;
+
+        log.debug("Temps d'exécution : " + executionTime + " secondes");
+        return result;
+    }
+}