You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@reef.apache.org by jo...@apache.org on 2015/11/16 01:34:52 UTC

incubator-reef git commit: [REEF-949] Remove unnecessary TODO comment in Mesos-runtime module

Repository: incubator-reef
Updated Branches:
  refs/heads/master 0d5a006ad -> fc5d9079b


[REEF-949] Remove unnecessary TODO comment in Mesos-runtime module

Since the current code already submits its executing user to Mesos,
this PR simply removes a TODO comment to prevent further confusion.

JIRA:
  [REEF-949](https://issues.apache.org/jira/browse/REEF-949)

Pull request:
  This closes #643


Project: http://git-wip-us.apache.org/repos/asf/incubator-reef/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-reef/commit/fc5d9079
Tree: http://git-wip-us.apache.org/repos/asf/incubator-reef/tree/fc5d9079
Diff: http://git-wip-us.apache.org/repos/asf/incubator-reef/diff/fc5d9079

Branch: refs/heads/master
Commit: fc5d9079bb080120932da496635b8e1bcf9c7f54
Parents: 0d5a006
Author: Dongjoon Hyun <do...@apache.org>
Authored: Mon Nov 16 02:55:18 2015 +0900
Committer: John Yang <jo...@apache.org>
Committed: Mon Nov 16 09:33:18 2015 +0900

----------------------------------------------------------------------
 .../java/org/apache/reef/runtime/mesos/driver/REEFScheduler.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-reef/blob/fc5d9079/lang/java/reef-runtime-mesos/src/main/java/org/apache/reef/runtime/mesos/driver/REEFScheduler.java
----------------------------------------------------------------------
diff --git a/lang/java/reef-runtime-mesos/src/main/java/org/apache/reef/runtime/mesos/driver/REEFScheduler.java b/lang/java/reef-runtime-mesos/src/main/java/org/apache/reef/runtime/mesos/driver/REEFScheduler.java
index 2dd4063..ea039f3 100644
--- a/lang/java/reef-runtime-mesos/src/main/java/org/apache/reef/runtime/mesos/driver/REEFScheduler.java
+++ b/lang/java/reef-runtime-mesos/src/main/java/org/apache/reef/runtime/mesos/driver/REEFScheduler.java
@@ -135,7 +135,7 @@ final class REEFScheduler implements Scheduler {
     this.schedulerDriverEStage = schedulerDriverEStage;
 
     final Protos.FrameworkInfo frameworkInfo = Protos.FrameworkInfo.newBuilder()
-        .setUser("") // TODO: make it configurable.
+        .setUser("")
         .setName(REEF_JOB_NAME_PREFIX + jobIdentifier)
         .build();
     this.mesosMaster = new MesosSchedulerDriver(this, frameworkInfo, masterIp);