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 2015/01/06 17:44:58 UTC

svn commit: r1649871 - in /uima/sandbox/uima-ducc/trunk/uima-ducc-user/src/main/java/org/apache/uima/ducc/user/jp: UimaASProcessContainer.java UimaProcessContainer.java

Author: cwiklik
Date: Tue Jan  6 16:44:57 2015
New Revision: 1649871

URL: http://svn.apache.org/r1649871
Log:
UIMA-4060 improved logging to catch initialization errors in uima and uima-as containers

Modified:
    uima/sandbox/uima-ducc/trunk/uima-ducc-user/src/main/java/org/apache/uima/ducc/user/jp/UimaASProcessContainer.java
    uima/sandbox/uima-ducc/trunk/uima-ducc-user/src/main/java/org/apache/uima/ducc/user/jp/UimaProcessContainer.java

Modified: uima/sandbox/uima-ducc/trunk/uima-ducc-user/src/main/java/org/apache/uima/ducc/user/jp/UimaASProcessContainer.java
URL: http://svn.apache.org/viewvc/uima/sandbox/uima-ducc/trunk/uima-ducc-user/src/main/java/org/apache/uima/ducc/user/jp/UimaASProcessContainer.java?rev=1649871&r1=1649870&r2=1649871&view=diff
==============================================================================
--- uima/sandbox/uima-ducc/trunk/uima-ducc-user/src/main/java/org/apache/uima/ducc/user/jp/UimaASProcessContainer.java (original)
+++ uima/sandbox/uima-ducc/trunk/uima-ducc-user/src/main/java/org/apache/uima/ducc/user/jp/UimaASProcessContainer.java Tue Jan  6 16:44:57 2015
@@ -96,26 +96,35 @@ public class UimaASProcessContainer impl
 	public void deploy(String duccHome) throws Exception {
 		// deploy single instance of embedded broker
 		synchronized( UimaASProcessContainer.class) {
-			if ( brokerInstance == null ) {
-				System.out.println("UIMA-AS Version::"+UimaAsVersion.getFullVersionString());
-				// isolate broker by loading it in its own Class Loader
-				// Sets the brokerInstance
-				deployBroker(duccHome);
-				
-				// Broker is running 
-				brokerRunning = true;
-				// create a shared instance of UIMA-AS client
-				uimaASClient = new BaseUIMAAsynchronousEngine_impl();
+			try {
+				if ( brokerInstance == null ) {
+					System.out.println("UIMA-AS Version::"+UimaAsVersion.getFullVersionString());
+					// isolate broker by loading it in its own Class Loader
+					// Sets the brokerInstance
+					deployBroker(duccHome);
+					
+					// Broker is running 
+					brokerRunning = true;
+					// create a shared instance of UIMA-AS client
+					uimaASClient = new BaseUIMAAsynchronousEngine_impl();
 
-				int i = 0;
-				// Deploy UIMA-AS services
-				for (String dd : deploymentDescriptors) {
-					// keep the container id so that we can un-deploy it when shutting
-					// down
-					ids[i] = deployService(dd);
+					int i = 0;
+					// Deploy UIMA-AS services
+					for (String dd : deploymentDescriptors) {
+						// keep the container id so that we can un-deploy it when shutting
+						// down
+						ids[i] = deployService(dd);
+					}
+					// send GetMeta to UIMA-AS service and wait for a reply
+					initializeUimaAsClient(endpointName);
 				}
-				// send GetMeta to UIMA-AS service and wait for a reply
-				initializeUimaAsClient(endpointName);
+				
+			} catch ( Throwable e) {
+				Logger logger = UIMAFramework.getLogger();
+				logger.log(Level.WARNING, "UimaProcessContainer", e);
+				e.printStackTrace();
+				throw new RuntimeException(e);
+
 			}
 			//	Pin thread to its own CAS serializer
 			serializerMap.put( Thread.currentThread().getId(), new UimaSerializer());

Modified: uima/sandbox/uima-ducc/trunk/uima-ducc-user/src/main/java/org/apache/uima/ducc/user/jp/UimaProcessContainer.java
URL: http://svn.apache.org/viewvc/uima/sandbox/uima-ducc/trunk/uima-ducc-user/src/main/java/org/apache/uima/ducc/user/jp/UimaProcessContainer.java?rev=1649871&r1=1649870&r2=1649871&view=diff
==============================================================================
--- uima/sandbox/uima-ducc/trunk/uima-ducc-user/src/main/java/org/apache/uima/ducc/user/jp/UimaProcessContainer.java (original)
+++ uima/sandbox/uima-ducc/trunk/uima-ducc-user/src/main/java/org/apache/uima/ducc/user/jp/UimaProcessContainer.java Tue Jan  6 16:44:57 2015
@@ -247,8 +247,13 @@ public class UimaProcessContainer implem
 				}
 			}
 
-		} catch (Exception e) {
+		} catch (Throwable e) {
 			latch.countDown();
+			Logger logger = UIMAFramework.getLogger();
+			logger.log(Level.WARNING, "UimaProcessContainer", e);
+			e.printStackTrace();
+			throw new RuntimeException(e);
+
 		} finally {
 			sharedInitSemaphore.release();