ping will always start a transaction

Author: sterlp

core/src/main/java/org/sterl/spring/persistent_tasks/scheduler/SchedulerService.java

@@ -70,11 +70,14 @@ public void shutdownNow() {
     }
 
     public SchedulerEntity pingRegistry() {
-        var result = editSchedulerStatus.checkinToRegistry(name);
-        result.setRunnungTasks(taskExecutor.getRunningTasks());
-        result.setTasksSlotCount(taskExecutor.getMaxThreads());
-        log.debug("Ping {}", result);
-        return result;
+        // using trx template to ensure the TRX is started if we use this method internally
+        return trx.execute(t -> {
+            var result = editSchedulerStatus.checkinToRegistry(name);
+            result.setRunnungTasks(taskExecutor.getRunningTasks());
+            result.setTasksSlotCount(taskExecutor.getMaxThreads());
+            log.debug("Ping {}", result);
+            return result;
+        });
     }
 
     public SchedulerEntity getScheduler() {
@@ -108,7 +111,7 @@ public List<Future<TriggerKey>> triggerNextTasks(OffsetDateTime timeDue) {
             triggers = Collections.emptyList();
         }
         var result = taskExecutor.submit(triggers);
-        trx.executeWithoutResult(t -> this.pingRegistry());
+        pingRegistry();
         return result;
     }