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 2015/08/11 08:39:21 UTC
svn commit: r1695219 - in /hama/trunk:
core/src/main/java/org/apache/hama/bsp/sync/ZooKeeperSyncClientImpl.java
yarn/src/main/java/org/apache/hama/bsp/ApplicationMaster.java
yarn/src/main/java/org/apache/hama/bsp/YARNBSPJob.java
Author: minho
Date: Tue Aug 11 06:39:21 2015
New Revision: 1695219
URL: http://svn.apache.org/r1695219
Log:
HAMA-963. Fix the problem that occurs ArrayIndexOutOfBoundsException
Modified:
hama/trunk/core/src/main/java/org/apache/hama/bsp/sync/ZooKeeperSyncClientImpl.java
hama/trunk/yarn/src/main/java/org/apache/hama/bsp/ApplicationMaster.java
hama/trunk/yarn/src/main/java/org/apache/hama/bsp/YARNBSPJob.java
Modified: hama/trunk/core/src/main/java/org/apache/hama/bsp/sync/ZooKeeperSyncClientImpl.java
URL: http://svn.apache.org/viewvc/hama/trunk/core/src/main/java/org/apache/hama/bsp/sync/ZooKeeperSyncClientImpl.java?rev=1695219&r1=1695218&r2=1695219&view=diff
==============================================================================
--- hama/trunk/core/src/main/java/org/apache/hama/bsp/sync/ZooKeeperSyncClientImpl.java (original)
+++ hama/trunk/core/src/main/java/org/apache/hama/bsp/sync/ZooKeeperSyncClientImpl.java Tue Aug 11 06:39:21 2015
@@ -241,7 +241,6 @@ public class ZooKeeperSyncClientImpl ext
String jobRegisterKey = constructKey(jobId, "peers");
Stat stat = null;
- LOG.info("TaskAttemptID : " + taskId);
while (stat != null) {
try {
stat = zk.exists(jobRegisterKey, false);
Modified: hama/trunk/yarn/src/main/java/org/apache/hama/bsp/ApplicationMaster.java
URL: http://svn.apache.org/viewvc/hama/trunk/yarn/src/main/java/org/apache/hama/bsp/ApplicationMaster.java?rev=1695219&r1=1695218&r2=1695219&view=diff
==============================================================================
--- hama/trunk/yarn/src/main/java/org/apache/hama/bsp/ApplicationMaster.java (original)
+++ hama/trunk/yarn/src/main/java/org/apache/hama/bsp/ApplicationMaster.java Tue Aug 11 06:39:21 2015
@@ -85,7 +85,10 @@ public class ApplicationMaster implement
private Counters globalCounter = new Counters();
private BSPJobClient.RawSplit[] splits;
+ // Hama job id
private BSPJobID jobId;
+ // Partiion id
+ private static AtomicInteger ai = new AtomicInteger(-1);
// SyncServer for Zookeeper
private SyncServer syncServer;
@@ -723,7 +726,7 @@ public class ApplicationMaster implement
vargs.add(BSPRunner.class.getCanonicalName());
vargs.add(jobId.getJtIdentifier());
- vargs.add(Long.toString(container.getId().getContainerId()));
+ vargs.add(Integer.toString(ai.incrementAndGet()));
vargs.add(new Path(jobFile).makeQualified(fs.getUri(),
fs.getWorkingDirectory()).toString());
@@ -945,7 +948,6 @@ public class ApplicationMaster implement
public Task getTask(TaskAttemptID taskid) throws IOException {
BSPJobClient.RawSplit assignedSplit = null;
String splitName = NullInputFormat.NullInputSplit.class.getName();
- // String splitName = NullInputSplit.class.getCanonicalName();
if (splits != null) {
assignedSplit = splits[taskid.id];
splitName = assignedSplit.getClassName();
Modified: hama/trunk/yarn/src/main/java/org/apache/hama/bsp/YARNBSPJob.java
URL: http://svn.apache.org/viewvc/hama/trunk/yarn/src/main/java/org/apache/hama/bsp/YARNBSPJob.java?rev=1695219&r1=1695218&r2=1695219&view=diff
==============================================================================
--- hama/trunk/yarn/src/main/java/org/apache/hama/bsp/YARNBSPJob.java (original)
+++ hama/trunk/yarn/src/main/java/org/apache/hama/bsp/YARNBSPJob.java Tue Aug 11 06:39:21 2015
@@ -74,6 +74,7 @@ public class YARNBSPJob extends BSPJob {
public void submit() throws IOException, InterruptedException {
// If Constants.MAX_TASKS_PER_JOB is null, calculates the max tasks based on resource status.
this.getConfiguration().setInt(Constants.MAX_TASKS_PER_JOB, getMaxTasks());
+
LOG.debug("MaxTasks: " + this.getConfiguration().get(Constants.MAX_TASKS_PER_JOB));
RunningJob submitJobInternal = submitClient.submitJobInternal(this,