optimise logging by avoiding cflowbelow and not logging entry/exit for getters, setters, and in repository

1 files changed, 9 insertions, 5 deletions

diff --git a/org.eclipse.virgo.medic/src/main/java/org/eclipse/virgo/medic/log/EntryExitTrace.aj b/org.eclipse.virgo.medic/src/main/java/org/eclipse/virgo/medic/log/EntryExitTrace.aj
index 5e4d2c0..7b0b1d8 100644
--- a/org.eclipse.virgo.medic/src/main/java/org/eclipse/virgo/medic/log/EntryExitTrace.aj
+++ b/org.eclipse.virgo.medic/src/main/java/org/eclipse/virgo/medic/log/EntryExitTrace.aj
@@ -27,19 +27,23 @@ public aspect EntryExitTrace pertypewithin(*) {
 	
 	private volatile Logger logger;
 	
-	pointcut performingEntryExitTrace() : cflowbelow(adviceexecution() && within(EntryExitTrace));
-
     pointcut medic() : within(org.eclipse.virgo.medic..*);
 
     pointcut util() : within(org.eclipse.virgo.util..*);
     
+    pointcut repository() : within(org.eclipse.virgo.repository..*);
+    
     pointcut logback() : within(ch.qos.logback..*) || within(org.slf4j.impl..*);
     
-    pointcut infoCandidate() : execution(public * *(..)) && !medic() && !util() && !logback() && !performingEntryExitTrace();
+    pointcut setter() : execution(* set*(..));
+    
+    pointcut getter() : execution(* get*(..));
+    
+    pointcut infoCandidate() : execution(public * *(..)) && !setter() && !getter() && !medic() && !util() && !repository() && !logback();
     
-    pointcut debugCandidate() : execution(!public !private * *(..)) && !medic() && !util() && !logback() && !performingEntryExitTrace();
+    pointcut debugCandidate() : execution(!public !private * *(..)) && !setter() && !getter() && !medic() && !util() && !repository() && !logback();
     
-    pointcut traceCandidate() : execution(private * *(..)) && !medic() && !util() && !logback() && !performingEntryExitTrace();
+    pointcut traceCandidate() : execution(private * *(..)) && !setter() && !getter() && !medic() && !util() && !repository() && !logback();
     
     before() : infoCandidate() {
         getLogger(thisJoinPointStaticPart).info("{} {}", ">", getSignature(thisJoinPointStaticPart));