You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@uima.apache.org by cw...@apache.org on 2009/10/27 14:36:29 UTC

svn commit: r830182 - in /incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae: error/ error/handler/ handler/input/

Author: cwiklik
Date: Tue Oct 27 13:36:29 2009
New Revision: 830182

URL: http://svn.apache.org/viewvc?rev=830182&view=rev
Log:
UIMA-1638 Removed printStackTrace()

Modified:
    incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/error/ErrorHandlerBase.java
    incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/error/handler/ProcessCasErrorHandler.java
    incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/handler/input/MetadataRequestHandler_impl.java
    incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/handler/input/MetadataResponseHandler_impl.java
    incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/handler/input/ProcessRequestHandler_impl.java
    incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/handler/input/ProcessResponseHandler.java

Modified: incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/error/ErrorHandlerBase.java
URL: http://svn.apache.org/viewvc/incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/error/ErrorHandlerBase.java?rev=830182&r1=830181&r2=830182&view=diff
==============================================================================
--- incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/error/ErrorHandlerBase.java (original)
+++ incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/error/ErrorHandlerBase.java Tue Oct 27 13:36:29 2009
@@ -221,7 +221,6 @@
           return true;
         } catch (Exception e) {
           anEndpoint.cancelTimer();
-          e.printStackTrace();
           if (UIMAFramework.getLogger(CLASS_NAME).isLoggable(Level.WARNING)) {
             UIMAFramework.getLogger(CLASS_NAME).logrb(Level.WARNING, getClass().getName(),
                     "retryLastCommand", UIMAEE_Constants.JMS_LOG_RESOURCE_BUNDLE,

Modified: incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/error/handler/ProcessCasErrorHandler.java
URL: http://svn.apache.org/viewvc/incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/error/handler/ProcessCasErrorHandler.java?rev=830182&r1=830181&r2=830182&view=diff
==============================================================================
--- incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/error/handler/ProcessCasErrorHandler.java (original)
+++ incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/error/handler/ProcessCasErrorHandler.java Tue Oct 27 13:36:29 2009
@@ -151,7 +151,6 @@
           }
         }
       } catch (Exception e) {
-        e.printStackTrace();
         if (UIMAFramework.getLogger(CLASS_NAME).isLoggable(Level.WARNING)) {
           UIMAFramework.getLogger(CLASS_NAME).logrb(Level.WARNING, getClass().getName(),
                   "sendExceptionToParent", UIMAEE_Constants.JMS_LOG_RESOURCE_BUNDLE,
@@ -447,8 +446,6 @@
           flowControllerContinueFlag = ((AggregateAnalysisEngineController) aController)
                   .continueOnError(casReferenceId, key, (Exception) t);
         } catch (Exception exc) {
-          exc.printStackTrace();
-
           if (UIMAFramework.getLogger(CLASS_NAME).isLoggable(Level.WARNING)) {
             UIMAFramework.getLogger(CLASS_NAME).logrb(Level.WARNING, getClass().getName(),
                     "handleError", UIMAEE_Constants.JMS_LOG_RESOURCE_BUNDLE,
@@ -558,7 +555,6 @@
             sendExceptionToClient(t, casReferenceId, endpoint, aController);
           }
         } catch (Exception e) {
-          e.printStackTrace();
           if (UIMAFramework.getLogger(CLASS_NAME).isLoggable(Level.WARNING)) {
             UIMAFramework.getLogger(CLASS_NAME).logrb(Level.WARNING, getClass().getName(),
                     "handleError", UIMAEE_Constants.JMS_LOG_RESOURCE_BUNDLE,
@@ -601,7 +597,6 @@
           }
         }
       } catch (Exception e) {
-        e.printStackTrace();
         if (UIMAFramework.getLogger(CLASS_NAME).isLoggable(Level.WARNING)) {
           UIMAFramework.getLogger(CLASS_NAME).logrb(Level.WARNING, getClass().getName(),
                   "handleError", UIMAEE_Constants.JMS_LOG_RESOURCE_BUNDLE,
@@ -615,7 +610,6 @@
           sendExceptionToClient(t, casReferenceId, endpoint, aController);
         }
       } catch (Exception e) {
-        e.printStackTrace();
         if (UIMAFramework.getLogger(CLASS_NAME).isLoggable(Level.WARNING)) {
           UIMAFramework.getLogger(CLASS_NAME).logrb(Level.WARNING, getClass().getName(),
                   "handleError", UIMAEE_Constants.JMS_LOG_RESOURCE_BUNDLE,
@@ -651,7 +645,6 @@
 
       aController.dropStats(casReferenceId, aController.getName());
     } catch (Exception e) {
-      e.printStackTrace();
       if (UIMAFramework.getLogger(CLASS_NAME).isLoggable(Level.WARNING)) {
         UIMAFramework.getLogger(CLASS_NAME).logrb(Level.WARNING, getClass().getName(),
                 "handleError", UIMAEE_Constants.JMS_LOG_RESOURCE_BUNDLE,

Modified: incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/handler/input/MetadataRequestHandler_impl.java
URL: http://svn.apache.org/viewvc/incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/handler/input/MetadataRequestHandler_impl.java?rev=830182&r1=830181&r2=830182&view=diff
==============================================================================
--- incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/handler/input/MetadataRequestHandler_impl.java (original)
+++ incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/handler/input/MetadataRequestHandler_impl.java Tue Oct 27 13:36:29 2009
@@ -85,7 +85,6 @@
           }
         }
       } catch (Exception e) {
-        e.printStackTrace();
         getController().getErrorHandlerChain().handle(e,
                 HandlerBase.populateErrorContext((MessageContext) anObjectToHandle),
                 getController());

Modified: incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/handler/input/MetadataResponseHandler_impl.java
URL: http://svn.apache.org/viewvc/incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/handler/input/MetadataResponseHandler_impl.java?rev=830182&r1=830181&r2=830182&view=diff
==============================================================================
--- incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/handler/input/MetadataResponseHandler_impl.java (original)
+++ incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/handler/input/MetadataResponseHandler_impl.java Tue Oct 27 13:36:29 2009
@@ -147,7 +147,6 @@
           }
         }
       } catch (Exception e) {
-        e.printStackTrace();
         getController().notifyListenersWithInitializationStatus(e);
         getController().getErrorHandlerChain().handle(e,
                 HandlerBase.populateErrorContext((MessageContext) anObjectToHandle),

Modified: incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/handler/input/ProcessRequestHandler_impl.java
URL: http://svn.apache.org/viewvc/incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/handler/input/ProcessRequestHandler_impl.java?rev=830182&r1=830181&r2=830182&view=diff
==============================================================================
--- incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/handler/input/ProcessRequestHandler_impl.java (original)
+++ incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/handler/input/ProcessRequestHandler_impl.java Tue Oct 27 13:36:29 2009
@@ -414,7 +414,11 @@
             getController().getOutputChannel().sendReply(AsynchAEMessage.ServiceInfo,
                     aMessageContext.getEndpoint(), casReferenceId);
           } catch (Exception e) {
-            e.printStackTrace();
+            if (UIMAFramework.getLogger(CLASS_NAME).isLoggable(Level.WARNING)) {
+              UIMAFramework.getLogger(CLASS_NAME).logrb(Level.WARNING, getClass().getName(),
+                      "handleProcessRequestFromRemoteClient", UIMAEE_Constants.JMS_LOG_RESOURCE_BUNDLE,
+                      "UIMAEE_exception__WARNING", new Object[] { e });
+            }
           }
         }
       }
@@ -489,7 +493,6 @@
         }
       }
     } catch (Exception e) {
-      e.printStackTrace();
       if (UIMAFramework.getLogger(CLASS_NAME).isLoggable(Level.WARNING)) {
         UIMAFramework.getLogger(CLASS_NAME).logrb(Level.WARNING, getClass().getName(),
                 "handleProcessRequestFromRemoteClient", UIMAEE_Constants.JMS_LOG_RESOURCE_BUNDLE,
@@ -572,7 +575,6 @@
                       UIMAEE_Constants.JMS_LOG_RESOURCE_BUNDLE, "UIMAEE_exception__WARNING",
                       new Object[] { e });
             }
-            e.printStackTrace();
             return;
           }
           // Safety check. The input Cas should not be null here
@@ -632,7 +634,7 @@
                 }
               }
             } catch (Exception e) {
-              e.printStackTrace();
+             
               if (UIMAFramework.getLogger(CLASS_NAME).isLoggable(Level.WARNING)) {
                 UIMAFramework.getLogger(CLASS_NAME).logrb(Level.WARNING, getClass().getName(),
                         "handleProcessRequestWithCASReference",
@@ -847,7 +849,11 @@
       getController().getOutputChannel().sendReply(AsynchAEMessage.Ping,
               aMessageContext.getEndpoint());
     } catch (Exception e) {
-      e.printStackTrace();
+      if (UIMAFramework.getLogger(CLASS_NAME).isLoggable(Level.WARNING)) {
+        UIMAFramework.getLogger(CLASS_NAME).logrb(Level.WARNING, getClass().getName(),
+                "handlePingRequest", UIMAEE_Constants.JMS_LOG_RESOURCE_BUNDLE,
+                "UIMAEE_exception__WARNING", new Object[] { e });
+      }
     }
 
   }
@@ -950,7 +956,11 @@
       // Not a Request nor Command. Delegate to the next handler in the chain
       super.delegate(messageContext);
     } catch (Exception e) {
-      e.printStackTrace();
+      if (UIMAFramework.getLogger(CLASS_NAME).isLoggable(Level.WARNING)) {
+        UIMAFramework.getLogger(CLASS_NAME).logrb(Level.WARNING, getClass().getName(),
+                "handle", UIMAEE_Constants.JMS_LOG_RESOURCE_BUNDLE,
+                "UIMAEE_exception__WARNING", new Object[] { e });
+      }
       getController().getErrorHandlerChain().handle(e,
               HandlerBase.populateErrorContext((MessageContext) anObjectToHandle), getController());
     }

Modified: incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/handler/input/ProcessResponseHandler.java
URL: http://svn.apache.org/viewvc/incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/handler/input/ProcessResponseHandler.java?rev=830182&r1=830181&r2=830182&view=diff
==============================================================================
--- incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/handler/input/ProcessResponseHandler.java (original)
+++ incubator/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/handler/input/ProcessResponseHandler.java Tue Oct 27 13:36:29 2009
@@ -316,7 +316,6 @@
         // The reply came late. The CAS was removed from the cache.
         return;
       }
-      e.printStackTrace();
       ErrorContext errorContext = new ErrorContext();
       errorContext.add(AsynchAEMessage.Command, AsynchAEMessage.Process);
       errorContext.add(AsynchAEMessage.CasReference, casReferenceId);
@@ -396,7 +395,6 @@
       }
     } catch (Exception e) {
 
-      e.printStackTrace();
       ErrorContext errorContext = new ErrorContext();
       errorContext.add(AsynchAEMessage.Command, AsynchAEMessage.Process);
       errorContext.add(AsynchAEMessage.CasReference, casReferenceId);
@@ -426,7 +424,11 @@
 
           }
         } catch (Exception e) {
-          e.printStackTrace();
+          if (UIMAFramework.getLogger(CLASS_NAME).isLoggable(Level.WARNING)) {
+            UIMAFramework.getLogger(CLASS_NAME).logrb(Level.WARNING, getClass().getName(),
+                    "handleProcessResponseWithCASReference", UIMAEE_Constants.JMS_LOG_RESOURCE_BUNDLE,
+                    "UIMAEE_exception__WARNING", new Object[] { e });
+          }
         }
       }
     }
@@ -536,7 +538,6 @@
                 getController());
       }
     } catch (Exception e) {
-      e.printStackTrace();
       ErrorContext errorContext = new ErrorContext();
       errorContext.add(AsynchAEMessage.Command, AsynchAEMessage.Process);
       errorContext.add(AsynchAEMessage.CasReference, casReferenceId);
@@ -554,7 +555,6 @@
                 .processCollectionCompleteReplyFromDelegate(delegateKey, true);
       }
     } catch (Exception e) {
-      e.printStackTrace();
       ErrorContext errorContext = new ErrorContext();
       errorContext.add(AsynchAEMessage.Command, AsynchAEMessage.CollectionProcessComplete);
       errorContext.add(AsynchAEMessage.Endpoint, aMessageContext.getEndpoint());
@@ -572,7 +572,11 @@
     try {
 
     } catch (Exception e) {
-      e.printStackTrace();
+      if (UIMAFramework.getLogger(CLASS_NAME).isLoggable(Level.WARNING)) {
+        UIMAFramework.getLogger(CLASS_NAME).logrb(Level.WARNING, getClass().getName(),
+                "handlePingReply", UIMAEE_Constants.JMS_LOG_RESOURCE_BUNDLE,
+                "UIMAEE_exception__WARNING", new Object[] { e });
+      }
     }
   }