You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@uima.apache.org by ch...@apache.org on 2015/03/26 18:17:15 UTC

svn commit: r1669372 - in /uima/sandbox/uima-ducc/trunk: uima-ducc-cli/src/test/java/org/apache/uima/ducc/cli/test/ uima-ducc-rm/src/main/java/org/apache/uima/ducc/rm/ uima-ducc-rm/src/main/java/org/apache/uima/ducc/rm/scheduler/

Author: challngr
Date: Thu Mar 26 17:17:14 2015
New Revision: 1669372

URL: http://svn.apache.org/r1669372
Log:
UIMA-4275 Clean up code to remove warnings.

Modified:
    uima/sandbox/uima-ducc/trunk/uima-ducc-cli/src/test/java/org/apache/uima/ducc/cli/test/ClassSeparation.java
    uima/sandbox/uima-ducc/trunk/uima-ducc-rm/src/main/java/org/apache/uima/ducc/rm/JobManagerConverter.java
    uima/sandbox/uima-ducc/trunk/uima-ducc-rm/src/main/java/org/apache/uima/ducc/rm/ResourceManagerComponent.java
    uima/sandbox/uima-ducc/trunk/uima-ducc-rm/src/main/java/org/apache/uima/ducc/rm/scheduler/NodePool.java

Modified: uima/sandbox/uima-ducc/trunk/uima-ducc-cli/src/test/java/org/apache/uima/ducc/cli/test/ClassSeparation.java
URL: http://svn.apache.org/viewvc/uima/sandbox/uima-ducc/trunk/uima-ducc-cli/src/test/java/org/apache/uima/ducc/cli/test/ClassSeparation.java?rev=1669372&r1=1669371&r2=1669372&view=diff
==============================================================================
--- uima/sandbox/uima-ducc/trunk/uima-ducc-cli/src/test/java/org/apache/uima/ducc/cli/test/ClassSeparation.java (original)
+++ uima/sandbox/uima-ducc/trunk/uima-ducc-cli/src/test/java/org/apache/uima/ducc/cli/test/ClassSeparation.java Thu Mar 26 17:17:14 2015
@@ -76,7 +76,8 @@ public class ClassSeparation
         }
 
         try {
-            XStream xs = new XStream(dd);
+            @SuppressWarnings("unused")
+			XStream xs = new XStream(dd);
         } catch ( IllegalStateException e ) {
             String msg = e.getMessage();
             if ( msg.equals("I am not XStream.") ) {

Modified: uima/sandbox/uima-ducc/trunk/uima-ducc-rm/src/main/java/org/apache/uima/ducc/rm/JobManagerConverter.java
URL: http://svn.apache.org/viewvc/uima/sandbox/uima-ducc/trunk/uima-ducc-rm/src/main/java/org/apache/uima/ducc/rm/JobManagerConverter.java?rev=1669372&r1=1669371&r2=1669372&view=diff
==============================================================================
--- uima/sandbox/uima-ducc/trunk/uima-ducc-rm/src/main/java/org/apache/uima/ducc/rm/JobManagerConverter.java (original)
+++ uima/sandbox/uima-ducc/trunk/uima-ducc-rm/src/main/java/org/apache/uima/ducc/rm/JobManagerConverter.java Thu Mar 26 17:17:14 2015
@@ -589,7 +589,7 @@ public class JobManagerConverter
         boolean status = true;        
         
         int max_processes = 0;
-       	int max_machines = 0;	
+       	// int max_machines = 0;	
         ResourceClass rescl = scheduler.getResourceClass(className);
 
         if ( rescl == null ) {
@@ -657,7 +657,7 @@ public class JobManagerConverter
                       break;
                       
                   case RESERVE:
-                      max_machines   = toInt(si.getSharesMax(), DEFAULT_INSTANCES);
+                      // max_machines   = toInt(si.getSharesMax(), DEFAULT_INSTANCES);
                       j.setMaxShares(max_processes);
                       break;
               }

Modified: uima/sandbox/uima-ducc/trunk/uima-ducc-rm/src/main/java/org/apache/uima/ducc/rm/ResourceManagerComponent.java
URL: http://svn.apache.org/viewvc/uima/sandbox/uima-ducc/trunk/uima-ducc-rm/src/main/java/org/apache/uima/ducc/rm/ResourceManagerComponent.java?rev=1669372&r1=1669371&r2=1669372&view=diff
==============================================================================
--- uima/sandbox/uima-ducc/trunk/uima-ducc-rm/src/main/java/org/apache/uima/ducc/rm/ResourceManagerComponent.java (original)
+++ uima/sandbox/uima-ducc/trunk/uima-ducc-rm/src/main/java/org/apache/uima/ducc/rm/ResourceManagerComponent.java Thu Mar 26 17:17:14 2015
@@ -165,7 +165,6 @@ public class ResourceManagerComponent
                     if ( ! validateAdministrator(dae) ) {
                         reply = new RmAdminReply("Not authorized");
                     } else {
-                        RmAdminReconfigure rc = (RmAdminReconfigure) body;            	 
                         reply = new RmAdminReply(scheduler.reconfigure());
                     }
                 } else

Modified: uima/sandbox/uima-ducc/trunk/uima-ducc-rm/src/main/java/org/apache/uima/ducc/rm/scheduler/NodePool.java
URL: http://svn.apache.org/viewvc/uima/sandbox/uima-ducc/trunk/uima-ducc-rm/src/main/java/org/apache/uima/ducc/rm/scheduler/NodePool.java?rev=1669372&r1=1669371&r2=1669372&view=diff
==============================================================================
--- uima/sandbox/uima-ducc/trunk/uima-ducc-rm/src/main/java/org/apache/uima/ducc/rm/scheduler/NodePool.java (original)
+++ uima/sandbox/uima-ducc/trunk/uima-ducc-rm/src/main/java/org/apache/uima/ducc/rm/scheduler/NodePool.java Thu Mar 26 17:17:14 2015
@@ -1156,7 +1156,6 @@ class NodePool
         Collections.sort(machs, new MachineByAscendingOrderSorter());
 
         Iterator<Machine> iter = machs.iterator();
-        ArrayList<Machine> pables = new ArrayList<Machine>();
         
         while ( iter.hasNext() && (given < needed) ) {
             Machine m = iter.next();