You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hama.apache.org by mi...@apache.org on 2013/11/26 13:48:35 UTC

svn commit: r1545638 - in /hama/trunk: CHANGES.txt core/src/main/java/org/apache/hama/bsp/BSPJobClient.java

Author: millecker
Date: Tue Nov 26 12:48:35 2013
New Revision: 1545638

URL: http://svn.apache.org/r1545638
Log:
HAMA-812: In local mode BSPJobClient.close throws java.lang.IllegalArgumentException: not a proxy instance

Modified:
    hama/trunk/CHANGES.txt
    hama/trunk/core/src/main/java/org/apache/hama/bsp/BSPJobClient.java

Modified: hama/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hama/trunk/CHANGES.txt?rev=1545638&r1=1545637&r2=1545638&view=diff
==============================================================================
--- hama/trunk/CHANGES.txt (original)
+++ hama/trunk/CHANGES.txt Tue Nov 26 12:48:35 2013
@@ -8,6 +8,7 @@ Release 0.7.0 (unreleased changes)
 
   BUG FIXES
 
+   HAMA-812: In local mode BSPJobClient.close throws Exception (Martin Illecker)
    HAMA-821: Fix bugs in KMeans example and make output more readable (edwardyoon)
 
   IMPROVEMENTS

Modified: hama/trunk/core/src/main/java/org/apache/hama/bsp/BSPJobClient.java
URL: http://svn.apache.org/viewvc/hama/trunk/core/src/main/java/org/apache/hama/bsp/BSPJobClient.java?rev=1545638&r1=1545637&r2=1545638&view=diff
==============================================================================
--- hama/trunk/core/src/main/java/org/apache/hama/bsp/BSPJobClient.java (original)
+++ hama/trunk/core/src/main/java/org/apache/hama/bsp/BSPJobClient.java Tue Nov 26 12:48:35 2013
@@ -241,7 +241,10 @@ public class BSPJobClient extends Config
    * Close the <code>JobClient</code>.
    */
   public synchronized void close() throws IOException {
-    RPC.stopProxy(jobSubmitClient);
+    String masterAdress = this.getConf().get("bsp.master.address");
+    if (masterAdress != null && !masterAdress.equals("local")) {
+      RPC.stopProxy(jobSubmitClient);
+    }
   }
 
   /**