[JBPM-10209] Gonzalos comments

kiegroup · Nov 15, 2023 · d95b41f · d95b41f
1 parent e497bc9
commit d95b41f
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/...bpm-services-ejb-timer/src/main/java/org/jbpm/services/ejb/timer/EjbSchedulerService.java b/...bpm-services-ejb-timer/src/main/java/org/jbpm/services/ejb/timer/EjbSchedulerService.java
@@ -98,12 +98,15 @@ public JobHandle scheduleJob(Job job, JobContext ctx, Trigger trigger) {
 
     @Override
     public boolean removeJob(JobHandle jobHandle) {
-        final Timer ejbTimer = getEjbTimer(getTimerMappinInfo(((EjbGlobalJobHandle) jobHandle).getUuid()));
+        String uuid = ((EjbGlobalJobHandle) jobHandle).getUuid();
+        final Timer ejbTimer = getEjbTimer(getTimerMappinInfo(uuid));
         if (TRANSACTIONAL && ejbTimer == null) {
-            // this situation needs to be avoided as it should not happen
+            logger.warn("EJB timer is null for uuid {} and transactional flag is enabled", uuid);
             return false;
         }
-        return scheduler.removeJob(jobHandle, ejbTimer);
+        boolean result = scheduler.removeJob(jobHandle, ejbTimer);
+        logger.debug("Remove job returned {}", result);
+        return result;
     }
 
     private TimerJobInstance getTimerJobInstance (String uuid) {