Improve executions and system calls

[gpl/argeo-slc.git] / runtime / org.argeo.slc.core / src / main / java / org / argeo / slc / core / execution / ExecutionAspect.java

diff --git a/runtime/org.argeo.slc.core/src/main/java/org/argeo/slc/core/execution/ExecutionAspect.java b/runtime/org.argeo.slc.core/src/main/java/org/argeo/slc/core/execution/ExecutionAspect.java
index 0f091fcb8db2fb98eedfea434a3b717c81518f04..d31c14847a79bd8b73ffc7c292938e3ac7966781 100644 (file)
--- a/runtime/org.argeo.slc.core/src/main/java/org/argeo/slc/core/execution/ExecutionAspect.java
+++ b/runtime/org.argeo.slc.core/src/main/java/org/argeo/slc/core/execution/ExecutionAspect.java
@@ -19,9 +19,10 @@ public class ExecutionAspect {
 
        @Around("flowExecution()")
        public void aroundFlow(ProceedingJoinPoint pjp) throws Throwable {
-               // IMPORTANT: Make sure that the execution context is called before the execution stack
+               // IMPORTANT: Make sure that the execution context is called before the
+               // execution stack
                executionContext.getUuid();
-               
+
                ExecutionFlow executionFlow = (ExecutionFlow) pjp.getTarget();
                executionStack.enterFlow(executionFlow);
                executionContext.setVariable(ExecutionContext.VAR_FLOW_ID,
@@ -39,6 +40,16 @@ public class ExecutionAspect {
                executionStack.leaveFlow(executionFlow);
        }
 
+       @Around("runnableExecution()")
+       public void aroundRunnable(ProceedingJoinPoint pjp) throws Throwable {
+               ExecutionFlow executionFlow = (ExecutionFlow) pjp.getTarget();
+               Runnable runnable = (Runnable) pjp.getArgs()[0];
+               if (log.isDebugEnabled())
+                       logRunnableExecution(executionFlow, runnable);
+               // Actually execute the runnable
+               pjp.proceed();
+       }
+
        @Around("getVariable()")
        public Object aroundGetVariable(ProceedingJoinPoint pjp) throws Throwable {
                Object obj = pjp.proceed();
@@ -55,6 +66,10 @@ public class ExecutionAspect {
        public void flowExecution() {
        }
 
+       @Pointcut("execution(void org.argeo.slc.execution.ExecutionFlow.doExecuteRunnable(..))")
+       public void runnableExecution() {
+       }
+
        @Pointcut("execution(* org.argeo.slc.execution.ExecutionContext.getVariable(..))")
        public void getVariable() {
        }
@@ -74,6 +89,13 @@ public class ExecutionAspect {
                                + stackSize);
        }
 
+       protected void logRunnableExecution(ExecutionFlow executionFlow,
+                       Runnable runnable) {
+               Integer stackSize = executionStack.getStackSize();
+               log.debug(depthSpaces(stackSize + 1)
+                               + runnable.getClass().getSimpleName() + " in " + executionFlow);
+       }
+
        private String depthSpaces(int depth) {
                StringBuffer buf = new StringBuffer(depth * 2);
                for (int i = 0; i < depth; i++)