You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@cocoon.apache.org by bd...@apache.org on 2003/05/02 15:10:23 UTC
cvs commit: cocoon-2.1/src/java/org/apache/cocoon/components/pipeline/impl AbstractCachingProcessingPipeline.java
bdelacretaz 2003/05/02 06:10:23
Modified: src/java/org/apache/cocoon/components/pipeline/impl
AbstractCachingProcessingPipeline.java
Log:
debugging log messages added to trace caching decisions
Revision Changes Path
1.4 +34 -9 cocoon-2.1/src/java/org/apache/cocoon/components/pipeline/impl/AbstractCachingProcessingPipeline.java
Index: AbstractCachingProcessingPipeline.java
===================================================================
RCS file: /home/cvs/cocoon-2.1/src/java/org/apache/cocoon/components/pipeline/impl/AbstractCachingProcessingPipeline.java,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -r1.3 -r1.4
--- AbstractCachingProcessingPipeline.java 20 Mar 2003 15:04:16 -0000 1.3
+++ AbstractCachingProcessingPipeline.java 2 May 2003 13:10:23 -0000 1.4
@@ -234,7 +234,7 @@
} else {
if (this.getLogger().isDebugEnabled() && this.toCacheKey != null) {
- this.getLogger().debug("Caching content for further requests of '" + environment.getURI() + "' using key " + this.toCacheKey);
+ this.getLogger().debug("processXMLPipeline: caching content for further requests of '" + environment.getURI() + "' using key " + this.toCacheKey);
}
try {
OutputStream os = null;
@@ -559,21 +559,29 @@
if ( !isValid ) {
responseIsValid = false;
// update validity
- if (validity == null)
+ if (validity == null) {
responseIsUsable = false;
- } else {
+ if (this.getLogger().isDebugEnabled()) {
+ this.getLogger().debug("validatePipeline: responseIsUsable is false, valid==" + valid + " at index " + i);
+ }
+ } else {
+ if (this.getLogger().isDebugEnabled()) {
+ this.getLogger().debug("validatePipeline: responseIsValid is false due to " + validity);
+ }
+ }
+ } else {
i++;
}
}
if ( responseIsValid ) {
if (this.getLogger().isDebugEnabled()) {
- this.getLogger().debug("Using valid cached content for '" + environment.getURI() + "'.");
+ this.getLogger().debug("validatePipeline: using valid cached content for '" + environment.getURI() + "'.");
}
// we are valid, ok that's it
this.cachedResponse = response.getResponse();
} else {
if (this.getLogger().isDebugEnabled()) {
- this.getLogger().debug("Cached content is invalid for '" + environment.getURI() + "'.");
+ this.getLogger().debug("validatePipeline: cached content is invalid for '" + environment.getURI() + "'.");
}
// we are not valid!
@@ -747,7 +755,7 @@
if (valid) {
if (this.getLogger().isDebugEnabled()) {
- this.getLogger().debug("Using valid cached content for '" + environment.getURI() + "'.");
+ this.getLogger().debug("processReader: using valid cached content for '" + environment.getURI() + "'.");
}
byte[] response = cachedObject.getResponse();
if (response.length > 0) {
@@ -758,7 +766,7 @@
}
} else {
if (this.getLogger().isDebugEnabled()) {
- this.getLogger().debug("Cached content is invalid for '" + environment.getURI() + "'.");
+ this.getLogger().debug("processReader: cached content is invalid for '" + environment.getURI() + "'.");
}
// remove invalid cached object
this.cache.remove(pcKey);
@@ -770,7 +778,7 @@
if ( pcKey != null ) {
if (this.getLogger().isDebugEnabled()) {
- this.getLogger().debug("Caching content for further requests of '" + environment.getURI() + "'.");
+ this.getLogger().debug("processReader: caching content for further requests of '" + environment.getURI() + "'.");
}
if (readerValidity == null) {
if (isCacheableProcessingComponent) {
@@ -862,28 +870,45 @@
*/
SourceValidity getValidityForInternalPipeline(int index) {
final SourceValidity validity;
+
+ // if debugging try to tell why something is not cacheable
+ final boolean debug = this.getLogger().isDebugEnabled();
+ String msg = null;
+ if(debug) msg = "getValidityForInternalPipeline(" + index + "): ";
+
if (index == 0) {
// test generator
if (this.generatorIsCacheableProcessingComponent) {
validity = ((CacheableProcessingComponent)super.generator).getValidity();
+ if(debug) msg += "generator: using getValidity";
} else {
validity = CacheValidityToSourceValidity.createValidity(((Cacheable)super.generator).generateValidity());
+ if(debug) msg += "generator: using generateValidity";
}
} else if (index <= firstNotCacheableTransformerIndex) {
// test transformer
final Transformer trans = (Transformer)super.transformers.get(index-1);
if (this.transformerIsCacheableProcessingComponent[index-1]) {
validity = ((CacheableProcessingComponent)trans).getValidity();
+ if(debug) msg += "transformer: using getValidity";
} else {
validity = CacheValidityToSourceValidity.createValidity(((Cacheable)trans).generateValidity());
+ if(debug) msg += "transformer: using generateValidity";
}
} else {
// test serializer
if (this.serializerIsCacheableProcessingComponent) {
validity = ((CacheableProcessingComponent)super.serializer).getValidity();
+ if(debug) msg += "serializer: using getValidity";
} else {
validity = CacheValidityToSourceValidity.createValidity(((Cacheable)super.serializer).generateValidity());
+ if(debug) msg += "serializer: using generateValidity";
}
+ }
+
+ if(debug) {
+ msg += ", validity==" + validity;
+ this.getLogger().debug(msg);
}
return validity;
}