svn commit: r823193 - in /incubator/wink/trunk/wink-server/src/main/java/org/apache/wink/server: handlers/AbstractHandlersChain.java internal/handlers/FindResourceMethodHandler.java

[ng...@apache.org]

Author: ngallardo
Date: Thu Oct  8 14:38:31 2009
New Revision: 823193

URL: http://svn.apache.org/viewvc?rev=823193&view=rev
Log:
[WINK-211]

Additional logging for handler chains.  Thanks to Mike Rhienheimer for the patch.

Modified:
    incubator/wink/trunk/wink-server/src/main/java/org/apache/wink/server/handlers/AbstractHandlersChain.java
    incubator/wink/trunk/wink-server/src/main/java/org/apache/wink/server/internal/handlers/FindResourceMethodHandler.java

Modified: incubator/wink/trunk/wink-server/src/main/java/org/apache/wink/server/handlers/AbstractHandlersChain.java
URL: http://svn.apache.org/viewvc/incubator/wink/trunk/wink-server/src/main/java/org/apache/wink/server/handlers/AbstractHandlersChain.java?rev=823193&r1=823192&r2=823193&view=diff
==============================================================================
--- incubator/wink/trunk/wink-server/src/main/java/org/apache/wink/server/handlers/AbstractHandlersChain.java (original)
+++ incubator/wink/trunk/wink-server/src/main/java/org/apache/wink/server/handlers/AbstractHandlersChain.java Thu Oct  8 14:38:31 2009
@@ -23,8 +23,13 @@
 import java.util.LinkedList;
 import java.util.ListIterator;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 public abstract class AbstractHandlersChain<T extends Handler> implements HandlersChain, Cloneable {
 
+    private static Logger logger = LoggerFactory.getLogger(AbstractHandlersChain.class);
+    
     private LinkedList<T>   list     = new LinkedList<T>();
     private ListIterator<T> iterator = null;
 
@@ -52,6 +57,7 @@
             // get the next handler from the chain to handle
             T handler = iterator.next();
             // invoke the handler
+            logger.debug("Invoking handler: {}", handler.getClass().getName());
             handle(handler, context);
         } finally {
             // set the iterator back one handler on the chain so the same

Modified: incubator/wink/trunk/wink-server/src/main/java/org/apache/wink/server/internal/handlers/FindResourceMethodHandler.java
URL: http://svn.apache.org/viewvc/incubator/wink/trunk/wink-server/src/main/java/org/apache/wink/server/internal/handlers/FindResourceMethodHandler.java?rev=823193&r1=823192&r2=823193&view=diff
==============================================================================
--- incubator/wink/trunk/wink-server/src/main/java/org/apache/wink/server/internal/handlers/FindResourceMethodHandler.java (original)
+++ incubator/wink/trunk/wink-server/src/main/java/org/apache/wink/server/internal/handlers/FindResourceMethodHandler.java Thu Oct  8 14:38:31 2009
@@ -60,7 +60,7 @@
 
         // resource method
         if (resource.isExactMatch()) {
-            logger.debug("Root resource @Path matches exactly so finding root resource method");
+            logger.debug("Root resource @Path matches exactly so finding root resource method in {}", resource.getResourceClass().getName());
             handleResourceMethod(context, chain);
             return;
         }