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/14 18:36:32 UTC

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

Author: cwiklik
Date: Wed Jan 14 17:36:32 2015
New Revision: 1651741

URL: http://svn.apache.org/r1651741
Log:
UIMA-4066 disable performance metrics for DD jobs due to bugs in uima-as metrics collection

Modified:
    uima/sandbox/uima-ducc/trunk/uima-ducc-user/src/main/java/org/apache/uima/ducc/user/jp/UimaASProcessContainer.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=1651741&r1=1651740&r2=1651741&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 Wed Jan 14 17:36:32 2015
@@ -341,12 +341,23 @@ implements IProcessContainer {
 			serializerMap.get(Thread.currentThread().getId()).
 				deserializeCasFromXmi((String)xmi, cas, deserSharedData, true,
 					-1);
+			/*
+			 * The following code commented for now. Re-enable when uima-as
+			 * performance metric collection is fixed. There is a bug in 
+			 * the uima-as which causes metrics to be invalid.
 			List<AnalysisEnginePerformanceMetrics> perfMetrics = new ArrayList<AnalysisEnginePerformanceMetrics>();
+			*/
+			
 			// delegate processing to the UIMA-AS service and wait for a reply
-			uimaASClient.sendAndReceiveCAS(cas, perfMetrics);
+			uimaASClient.sendAndReceiveCAS(cas);//, perfMetrics);
 			// convert UIMA-AS metrics into properties so that we can return this
 			// data in a format which doesnt require UIMA-AS to digest
 			List<Properties> metricsList = new ArrayList<Properties>(); 
+			/*
+			 * The following code commented for now. Re-enable when uima-as
+			 * performance metric collection is fixed. There is a bug in 
+			 * the uima-as which causes metrics to be invalid.
+
 			for( AnalysisEnginePerformanceMetrics metrics : perfMetrics ) {
 				Properties p = new Properties();
 				p.setProperty("name", metrics.getName());
@@ -355,6 +366,7 @@ implements IProcessContainer {
 				p.setProperty("numProcessed",String.valueOf(metrics.getNumProcessed()) );
 				metricsList.add(p);
 			}
+			*/
 			return metricsList;
 		} catch( Throwable e ) {
 			lastError = e;