Hi Adrian, All, Should we really show the timing for all services? Maybe increasing from 50 to 75 or even 100 for the 1st case would be enough?
Jacques From: <adri...@apache.org> > Author: adrianc > Date: Wed Apr 3 07:57:24 2013 > New Revision: 1463863 > > URL: http://svn.apache.org/r1463863 > Log: > Log message cleanup in ServiceDispatcher.java. Removed confusing text about > services taking too long. > > Modified: > ofbiz/trunk/framework/service/src/org/ofbiz/service/ServiceDispatcher.java > > Modified: > ofbiz/trunk/framework/service/src/org/ofbiz/service/ServiceDispatcher.java > URL: > http://svn.apache.org/viewvc/ofbiz/trunk/framework/service/src/org/ofbiz/service/ServiceDispatcher.java?rev=1463863&r1=1463862&r2=1463863&view=diff > ============================================================================== > --- > ofbiz/trunk/framework/service/src/org/ofbiz/service/ServiceDispatcher.java > (original) > +++ > ofbiz/trunk/framework/service/src/org/ofbiz/service/ServiceDispatcher.java > Wed Apr 3 07:57:24 2013 > @@ -571,10 +571,8 @@ public class ServiceDispatcher { > rs.setEndStamp(); > > long timeToRun = System.currentTimeMillis() - serviceStartTime; > - if (Debug.timingOn() && timeToRun > 50) { > - Debug.logTiming("Slow sync service execution detected: service > [" + localName + "/" + modelService.name + "] finished in [" + timeToRun + "] > milliseconds", module); > - } else if (Debug.infoOn() && timeToRun > 200) { > - Debug.logInfo("Very slow sync service execution detected: > service [" + localName + "/" + modelService.name + "] finished in [" + > timeToRun + "] milliseconds", module); > + if (Debug.timingOn()) { > + Debug.logTiming("Sync service [" + localName + "/" + > modelService.name + "] finished in [" + timeToRun + "] milliseconds", module); > } > if ((Debug.verboseOn() || modelService.debug) && timeToRun > 50 && > !modelService.hideResultInLog) { > // Sanity check - some service results can be multiple MB in > size. Limit message size to 10K. > >
