You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@uima.apache.org by de...@apache.org on 2015/02/03 00:01:16 UTC

svn commit: r1656599 - in /uima/sandbox/uima-ducc/trunk/uima-ducc-container/src/main/java/org/apache/uima/ducc/container: common/Standardize.java jd/classload/ProxyJobDriverCollectionReader.java jd/classload/ProxyJobDriverUserError.java

Author: degenaro
Date: Mon Feb  2 23:01:15 2015
New Revision: 1656599

URL: http://svn.apache.org/r1656599
Log:
UIMA-4069 DUCC Job Driver (JD) system classpath

provide full stack trace in JD log when CR throws an exception

Modified:
    uima/sandbox/uima-ducc/trunk/uima-ducc-container/src/main/java/org/apache/uima/ducc/container/common/Standardize.java
    uima/sandbox/uima-ducc/trunk/uima-ducc-container/src/main/java/org/apache/uima/ducc/container/jd/classload/ProxyJobDriverCollectionReader.java
    uima/sandbox/uima-ducc/trunk/uima-ducc-container/src/main/java/org/apache/uima/ducc/container/jd/classload/ProxyJobDriverUserError.java

Modified: uima/sandbox/uima-ducc/trunk/uima-ducc-container/src/main/java/org/apache/uima/ducc/container/common/Standardize.java
URL: http://svn.apache.org/viewvc/uima/sandbox/uima-ducc/trunk/uima-ducc-container/src/main/java/org/apache/uima/ducc/container/common/Standardize.java?rev=1656599&r1=1656598&r2=1656599&view=diff
==============================================================================
--- uima/sandbox/uima-ducc/trunk/uima-ducc-container/src/main/java/org/apache/uima/ducc/container/common/Standardize.java (original)
+++ uima/sandbox/uima-ducc/trunk/uima-ducc-container/src/main/java/org/apache/uima/ducc/container/common/Standardize.java Mon Feb  2 23:01:15 2015
@@ -21,6 +21,9 @@ package org.apache.uima.ducc.container.c
 public class Standardize {
 
 	public enum Label {
+		classname,
+		exception,
+		instance,
 		id,
 		skip,
 		dispatched,

Modified: uima/sandbox/uima-ducc/trunk/uima-ducc-container/src/main/java/org/apache/uima/ducc/container/jd/classload/ProxyJobDriverCollectionReader.java
URL: http://svn.apache.org/viewvc/uima/sandbox/uima-ducc/trunk/uima-ducc-container/src/main/java/org/apache/uima/ducc/container/jd/classload/ProxyJobDriverCollectionReader.java?rev=1656599&r1=1656598&r2=1656599&view=diff
==============================================================================
--- uima/sandbox/uima-ducc/trunk/uima-ducc-container/src/main/java/org/apache/uima/ducc/container/jd/classload/ProxyJobDriverCollectionReader.java (original)
+++ uima/sandbox/uima-ducc/trunk/uima-ducc-container/src/main/java/org/apache/uima/ducc/container/jd/classload/ProxyJobDriverCollectionReader.java Mon Feb  2 23:01:15 2015
@@ -98,8 +98,8 @@ public class ProxyJobDriverCollectionRea
 			retVal = (Integer)method_getTotal.invoke(instance_JdUserCollectionReader, nullObjectArray);
 		} 
 		catch (Exception e) {
-			logger.error(location, ILogger.null_id, e);
 			ProxyJobDriverUserError.loggifyUserException(e);
+			logger.error(location, ILogger.null_id, e);
 			throw new JobDriverException(e);
 		}
 		return retVal;
@@ -123,8 +123,8 @@ public class ProxyJobDriverCollectionRea
 			}
 		} 
 		catch (Exception e) {
-			logger.error(location, ILogger.null_id, e);
 			ProxyJobDriverUserError.loggifyUserException(e);
+			logger.error(location, ILogger.null_id, e);
 			throw new JobDriverException(e);
 		}
 		return retVal;
@@ -170,8 +170,8 @@ public class ProxyJobDriverCollectionRea
 			method_getJdUserMetaCas = class_JdUserCollectionReader.getMethod(name_getJdUserMetaCas, nullClassArray);
 		} 
 		catch (Exception e) {
-			logger.error(location, ILogger.null_id, e);
 			ProxyJobDriverUserError.loggifyUserException(e);
+			logger.error(location, ILogger.null_id, e);
 			throw new JobDriverException(e);
 		}
 	}
@@ -206,8 +206,8 @@ public class ProxyJobDriverCollectionRea
 			logger.trace(location, ILogger.null_id, mb.toString());
 		} 
 		catch (Exception e) {
-			logger.error(location, ILogger.null_id, mb, e);
 			ProxyJobDriverUserError.loggifyUserException(e);
+			logger.error(location, ILogger.null_id, mb, e);
 			throw new JobDriverException(e);
 		}
 	}

Modified: uima/sandbox/uima-ducc/trunk/uima-ducc-container/src/main/java/org/apache/uima/ducc/container/jd/classload/ProxyJobDriverUserError.java
URL: http://svn.apache.org/viewvc/uima/sandbox/uima-ducc/trunk/uima-ducc-container/src/main/java/org/apache/uima/ducc/container/jd/classload/ProxyJobDriverUserError.java?rev=1656599&r1=1656598&r2=1656599&view=diff
==============================================================================
--- uima/sandbox/uima-ducc/trunk/uima-ducc-container/src/main/java/org/apache/uima/ducc/container/jd/classload/ProxyJobDriverUserError.java (original)
+++ uima/sandbox/uima-ducc/trunk/uima-ducc-container/src/main/java/org/apache/uima/ducc/container/jd/classload/ProxyJobDriverUserError.java Mon Feb  2 23:01:15 2015
@@ -18,6 +18,8 @@
 */
 package org.apache.uima.ducc.container.jd.classload;
 
+import org.apache.uima.ducc.container.common.MessageBuffer;
+import org.apache.uima.ducc.container.common.Standardize;
 import org.apache.uima.ducc.container.common.logger.IComponent;
 import org.apache.uima.ducc.container.common.logger.ILogger;
 import org.apache.uima.ducc.container.common.logger.Logger;
@@ -36,7 +38,22 @@ public class ProxyJobDriverUserError {
 				if(userException != null) {
 					logger.error(location, ILogger.null_id, userException);
 				}
+				else {
+					MessageBuffer mb = new MessageBuffer();
+					mb.append(Standardize.Label.instance.get()+null);
+					logger.debug(location, ILogger.null_id, mb);
+				}
+			}
+			else {
+				MessageBuffer mb = new MessageBuffer();
+				mb.append(Standardize.Label.classname.get()+e.getClass().getName());
+				logger.debug(location, ILogger.null_id, mb);
 			}
 		}
+		else {
+			MessageBuffer mb = new MessageBuffer();
+			mb.append(Standardize.Label.exception.get()+null);
+			logger.debug(location, ILogger.null_id, mb);
+		}
 	}
 }