Author: boisvert
Date: Thu Nov  9 12:35:17 2006
New Revision: 473067

URL: http://svn.apache.org/viewvc?view=rev&rev=473067
Log:
More specific error reporting

Modified:
    
incubator/ode/trunk/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/ProcessAndInstanceManagementImpl.java
    
incubator/ode/trunk/bpel-store/src/main/java/org/apache/ode/store/ProcessStoreImpl.java

Modified: 
incubator/ode/trunk/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/ProcessAndInstanceManagementImpl.java
URL: 
http://svn.apache.org/viewvc/incubator/ode/trunk/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/ProcessAndInstanceManagementImpl.java?view=diff&rev=473067&r1=473066&r2=473067
==============================================================================
--- 
incubator/ode/trunk/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/ProcessAndInstanceManagementImpl.java
 (original)
+++ 
incubator/ode/trunk/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/ProcessAndInstanceManagementImpl.java
 Thu Nov  9 12:35:17 2006
@@ -88,8 +88,7 @@
                 }
             });
         } catch (Exception e) {
-            __log.error("DbError", e);
-            throw new ProcessingException("DbError",e);
+            throw new ProcessingException("Exception while listing 
processes",e);
         }
 
         return ret;
@@ -146,8 +145,7 @@
         } catch (ManagementException me) {
             throw me;
         } catch (Exception e) {
-            __log.error("DbError", e);
-            throw new ProcessingException("DbError",e);
+            throw new ProcessingException("Exception while setting process 
property",e);
         }
 
         return ret;
@@ -171,8 +169,7 @@
         } catch (ManagementException me) {
             throw me;
         } catch (Exception e) {
-            __log.error("DbError", e);
-            throw new ProcessingException("DbError",e);
+            throw new ProcessingException("Exception while setting process 
property",e);
         }
 
         return ret;
@@ -194,8 +191,7 @@
                 }
             });
         } catch (Exception e) {
-            __log.error("DbError", e);
-            throw new ProcessingException("DbError",e);
+            throw new ProcessingException("Exception while listing 
instances",e);
         }
 
         return ret;
@@ -311,8 +307,7 @@
                 }
             });
         } catch (Exception e) {
-            __log.error("DbError",e);
-            throw new ProcessingException("DbError", e);
+            throw new ProcessingException("Exception during activity 
recovery", e);
         }
         return genInstanceInfoDocument(iid);
     }
@@ -333,8 +328,7 @@
                 }
             });
         } catch (Exception e) {
-            __log.error("DbError", e);
-            throw new ProcessingException("DbError",e);
+            throw new ProcessingException("Exception during instance 
deletion",e);
         }
 
         return ret;
@@ -444,8 +438,7 @@
                 }
             });
         } catch (Exception e) {
-            __log.error("DbError",e);
-            throw new ProcessingException("DbError", e);
+            throw new ProcessingException("Exception during instance 
retrieval", e);
         }
 
         return getDebugger(processId);
@@ -463,8 +456,7 @@
         } catch (ManagementException me) {
             throw me;
         } catch (Exception e) {
-            __log.error("DbError", e);
-            throw new ManagementException("DbError",e);
+            throw new ManagementException("Exception during database 
operation", e);
         }
     }
 
@@ -481,8 +473,7 @@
             // Passthrough.
             throw me;
         } catch (Exception ex) {
-            __log.error("DbError", ex);
-            throw new ManagementException("DbError",ex);
+            throw new ManagementException("Exception during database 
operation",ex);
         }
     }
 
@@ -503,8 +494,7 @@
         } catch (ManagementException me) {
             throw me;
         } catch (Exception e) {
-            __log.error("DbError", e);
-            throw new ProcessingException("UnexpectedEx",e);
+            throw new ProcessingException("Exception while retrieving process 
information",e);
         }
 
         return ret;
@@ -944,8 +934,7 @@
                 }
             });
         } catch (Exception e) {
-            __log.error("DbError", e);
-            throw new ProcessingException("DbError",e);
+            throw new ProcessingException("Exception while querying 
instances",e);
         }
 
         return ret;

Modified: 
incubator/ode/trunk/bpel-store/src/main/java/org/apache/ode/store/ProcessStoreImpl.java
URL: 
http://svn.apache.org/viewvc/incubator/ode/trunk/bpel-store/src/main/java/org/apache/ode/store/ProcessStoreImpl.java?view=diff&rev=473067&r1=473066&r2=473067
==============================================================================
--- 
incubator/ode/trunk/bpel-store/src/main/java/org/apache/ode/store/ProcessStoreImpl.java
 (original)
+++ 
incubator/ode/trunk/bpel-store/src/main/java/org/apache/ode/store/ProcessStoreImpl.java
 Thu Nov  9 12:35:17 2006
@@ -256,7 +256,6 @@
         } catch (RuntimeException ex) {
             throw ex;
         } catch (Exception ex) {
-            __log.error(__msgs.msgDbError(), ex);
             throw new BpelEngineException(ex);
         } finally {
             _mngmtLock.writeLock().unlock();
@@ -330,7 +329,6 @@
         } catch (RuntimeException ex) {
             throw ex;
         } catch (Exception ex) {
-            __log.error(__msgs.msgDbError(), ex);
             throw new BpelEngineException(ex);
         } finally {
             _mngmtLock.writeLock().unlock();
@@ -351,7 +349,6 @@
         } catch (RuntimeException ex) {
             throw ex;
         } catch (Exception ex) {
-            __log.error(__msgs.msgDbError(), ex);
             throw new BpelEngineException(ex);
         } finally {
             _mngmtLock.writeLock().unlock();
@@ -373,7 +370,6 @@
         } catch (RuntimeException ex) {
             throw ex;
         } catch (Exception ex) {
-            __log.error(__msgs.msgDbError(), ex);
             throw new BpelEngineException(ex);
         } finally {
             _mngmtLock.writeLock().unlock();
@@ -426,7 +422,6 @@
         } catch (RuntimeException ex) {
             throw ex;
         } catch (Exception ex) {
-            __log.error(__msgs.msgDbError(), ex);
             throw new BpelEngineException(ex);
         } finally {
             _mngmtLock.writeLock().unlock();
@@ -455,7 +450,6 @@
         } catch (RuntimeException ex) {
             throw ex;
         } catch (Exception ex) {
-            __log.error(__msgs.msgDbError(), ex);
             throw new BpelEngineException(ex);
         } finally {
             _mngmtLock.writeLock().unlock();


Reply via email to