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 2017/09/28 18:32:13 UTC

svn commit: r1810023 - /uima/uima-ducc/trunk/uima-ducc-agent/src/main/java/org/apache/uima/ducc/agent/processors/LinuxProcessMetricsProcessor.java

Author: cwiklik
Date: Thu Sep 28 18:32:13 2017
New Revision: 1810023

URL: http://svn.apache.org/viewvc?rev=1810023&view=rev
Log:
UIMA-5592 changed how agent determines if it needs to start collecting GC stats for remote process. Instead of testing for process type, it checks if jmx connect string has been provided by a child process

Modified:
    uima/uima-ducc/trunk/uima-ducc-agent/src/main/java/org/apache/uima/ducc/agent/processors/LinuxProcessMetricsProcessor.java

Modified: uima/uima-ducc/trunk/uima-ducc-agent/src/main/java/org/apache/uima/ducc/agent/processors/LinuxProcessMetricsProcessor.java
URL: http://svn.apache.org/viewvc/uima/uima-ducc/trunk/uima-ducc-agent/src/main/java/org/apache/uima/ducc/agent/processors/LinuxProcessMetricsProcessor.java?rev=1810023&r1=1810022&r2=1810023&view=diff
==============================================================================
--- uima/uima-ducc/trunk/uima-ducc-agent/src/main/java/org/apache/uima/ducc/agent/processors/LinuxProcessMetricsProcessor.java (original)
+++ uima/uima-ducc/trunk/uima-ducc-agent/src/main/java/org/apache/uima/ducc/agent/processors/LinuxProcessMetricsProcessor.java Thu Sep 28 18:32:13 2017
@@ -330,7 +330,9 @@ public class LinuxProcessMetricsProcesso
 	}
 
 	private ProcessGarbageCollectionStats getGCStats() throws Exception {
-		if (!process.getProcessType().equals(ProcessType.Pop)) {
+//		if (!process.getProcessType().equals(ProcessType.Pop)) {
+		if ( process.getProcessJmxUrl() != null
+				&& process.getProcessJmxUrl().trim().length() > 0 ) {
 			logger.info("LinuxProcessMetricsProcessor.getGCStats",	null, "Collecting GC Stats");
 			ProcessGarbageCollectionStats gcStats = gcStatsCollector
 					.collect();