CAUSEWAY-3751: suppress webRequestCycle if trivial
diff --git a/viewers/wicket/viewer/src/main/java/org/apache/causeway/viewer/wicket/viewer/integration/WebRequestCycleForCauseway.java b/viewers/wicket/viewer/src/main/java/org/apache/causeway/viewer/wicket/viewer/integration/WebRequestCycleForCauseway.java
index 5eb850f..d97b516 100644
--- a/viewers/wicket/viewer/src/main/java/org/apache/causeway/viewer/wicket/viewer/integration/WebRequestCycleForCauseway.java
+++ b/viewers/wicket/viewer/src/main/java/org/apache/causeway/viewer/wicket/viewer/integration/WebRequestCycleForCauseway.java
@@ -190,7 +190,6 @@ public synchronized void onBeginRequest(final RequestCycle requestCycle) {
if(log.isDebugEnabled()) {
timings.set(new Timing());
- log.debug("onBeginRequest ...");
}
}
@@ -278,11 +277,18 @@ public synchronized void onEndRequest(final RequestCycle requestCycle) {
if(log.isDebugEnabled()) {
val metricsServiceIfAny = getMetaModelContext().lookupService(MetricsService.class);
- if(metricsServiceIfAny.isPresent()) {
- val metricsService = metricsServiceIfAny.get();
- log.debug("onEndRequest took: {}ms numberEntitiesLoaded: {}, numberEntitiesDirtied: {}", timings.get().took(), metricsService.numberEntitiesLoaded(), metricsService.numberEntitiesDirtied());
- } else {
- log.debug("onEndRequest took: {}ms", timings.get().took());
+ long took = timings.get().took();
+ if(took > 50) { // avoid too much clutter
+ if(metricsServiceIfAny.isPresent()) {
+ val metricsService = metricsServiceIfAny.get();
+ int numberEntitiesLoaded = metricsService.numberEntitiesLoaded();
+ int numberEntitiesDirtied = metricsService.numberEntitiesDirtied();
+ if(numberEntitiesLoaded > 0 || numberEntitiesDirtied > 0) {
+ log.debug("onEndRequest took: {}ms numberEntitiesLoaded: {}, numberEntitiesDirtied: {}", took, numberEntitiesLoaded, numberEntitiesDirtied);
+ }
+ } else {
+ log.debug("onEndRequest took: {}ms", took);
+ }
}
}
