svn commit: r559998 - in /roller/trunk/apps/planet/src/java/org/apache/roller/planet/business: fetcher/FetcherException.java hibernate/HibernatePlanetImpl.java updater/SingleThreadedFeedUpdater.java

[ag...@apache.org]

Author: agilliland
Date: Thu Jul 26 14:40:22 2007
New Revision: 559998

URL: http://svn.apache.org/viewvc?view=rev&rev=559998
Log:
more fixes for logging.

Modified:
    roller/trunk/apps/planet/src/java/org/apache/roller/planet/business/fetcher/FetcherException.java
    roller/trunk/apps/planet/src/java/org/apache/roller/planet/business/hibernate/HibernatePlanetImpl.java
    roller/trunk/apps/planet/src/java/org/apache/roller/planet/business/updater/SingleThreadedFeedUpdater.java

Modified: roller/trunk/apps/planet/src/java/org/apache/roller/planet/business/fetcher/FetcherException.java
URL: http://svn.apache.org/viewvc/roller/trunk/apps/planet/src/java/org/apache/roller/planet/business/fetcher/FetcherException.java?view=diff&rev=559998&r1=559997&r2=559998
==============================================================================
--- roller/trunk/apps/planet/src/java/org/apache/roller/planet/business/fetcher/FetcherException.java (original)
+++ roller/trunk/apps/planet/src/java/org/apache/roller/planet/business/fetcher/FetcherException.java Thu Jul 26 14:40:22 2007
@@ -18,6 +18,8 @@
 
 package org.apache.roller.planet.business.fetcher;
 
+import java.io.PrintStream;
+import java.io.PrintWriter;
 import org.apache.roller.planet.PlanetException;
 
 
@@ -32,6 +34,54 @@
     
     public FetcherException(String msg, Throwable t) {
         super(msg, t);
+    }
+    
+    
+    /**
+     * Print stack trace for exception and for root cause exception if htere is one.
+     * @see java.lang.Throwable#printStackTrace()
+     */
+    @Override
+    public void printStackTrace() {
+        // just print our message since we know this exception should be wrapping
+        // a more detailed exception from whatever fetching solution is used
+        System.out.println(super.getMessage());
+        if (getRootCause() != null) {
+            System.out.println("--- ROOT CAUSE ---");
+            getRootCause().printStackTrace();
+        }
+    }
+    
+    
+    /**
+     * Print stack trace for exception and for root cause exception if htere is one.
+     * @param s Stream to print to.
+     */
+    @Override
+    public void printStackTrace(PrintStream s) {
+        // just print our message since we know this exception should be wrapping
+        // a more detailed exception from whatever fetching solution is used
+        s.println(super.getMessage());
+        if (getRootCause() != null) {
+            s.println("--- ROOT CAUSE ---");
+            getRootCause().printStackTrace(s);
+        }
+    }
+    
+    
+    /**
+     * Print stack trace for exception and for root cause exception if htere is one.
+     * @param s Writer to write to.
+     */
+    @Override
+    public void printStackTrace(PrintWriter s) {
+        // just print our message since we know this exception should be wrapping
+        // a more detailed exception from whatever fetching solution is used
+        s.println(super.getMessage());
+        if (getRootCause() != null) {
+            s.println("--- ROOT CAUSE ---");
+            getRootCause().printStackTrace(s);
+        }
     }
     
 }

Modified: roller/trunk/apps/planet/src/java/org/apache/roller/planet/business/hibernate/HibernatePlanetImpl.java
URL: http://svn.apache.org/viewvc/roller/trunk/apps/planet/src/java/org/apache/roller/planet/business/hibernate/HibernatePlanetImpl.java?view=diff&rev=559998&r1=559997&r2=559998
==============================================================================
--- roller/trunk/apps/planet/src/java/org/apache/roller/planet/business/hibernate/HibernatePlanetImpl.java (original)
+++ roller/trunk/apps/planet/src/java/org/apache/roller/planet/business/hibernate/HibernatePlanetImpl.java Thu Jul 26 14:40:22 2007
@@ -94,6 +94,9 @@
     
     @Override
     public void initialize() throws InitializationException {
+        
+        log.info("Initializing Roller Planet business tier");
+        
         getPropertiesManager().initialize();
         getPlanetManager().initialize();
         
@@ -104,6 +107,8 @@
         } catch(PlanetException ex) {
             throw new InitializationException("Error flushing after initialization", ex);
         }
+        
+        log.info("Roller Planet business tier successfully initialized");
     }
     
     

Modified: roller/trunk/apps/planet/src/java/org/apache/roller/planet/business/updater/SingleThreadedFeedUpdater.java
URL: http://svn.apache.org/viewvc/roller/trunk/apps/planet/src/java/org/apache/roller/planet/business/updater/SingleThreadedFeedUpdater.java?view=diff&rev=559998&r1=559997&r2=559998
==============================================================================
--- roller/trunk/apps/planet/src/java/org/apache/roller/planet/business/updater/SingleThreadedFeedUpdater.java (original)
+++ roller/trunk/apps/planet/src/java/org/apache/roller/planet/business/updater/SingleThreadedFeedUpdater.java Thu Jul 26 14:40:22 2007
@@ -185,7 +185,10 @@
                 updateSubscription(sub);
             } catch(UpdaterException ex) {
                 // do a little work to get at the source of the problem
-                Throwable cause = ex.getCause();
+                Throwable cause = ex;
+                if(ex.getRootCause() != null) {
+                    cause = ex.getRootCause();
+                }
                 if(cause.getCause() != null) {
                     cause = cause.getCause();
                 }