Repository: reef
Updated Branches:
  refs/heads/master a85effcb1 -> 17840f04c


[REEF-1699] Use the {{JAVA_HOME}} notation when starting REEF Evaluators on YARN

   * Use `{{JAVA_HOME}}` instead of locally-issued `.getenv()` call when 
launching REEF Evaluators on YARN
   * Implement a `.toString()` method for better logging

This fix is related to 
[REEF-1665](https://issues.apache.org/jira/browse/REEF-1665) that uses the same 
mechanism to launch the Driver

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

Closese #1215


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

Branch: refs/heads/master
Commit: 17840f04c4512682d03df959bf1cd8a995128286
Parents: a85effc
Author: Sergiy Matusevych <[email protected]>
Authored: Wed Jan 11 17:59:23 2017 -0800
Committer: Yunseong Lee <[email protected]>
Committed: Fri Jan 13 02:23:07 2017 +0900

----------------------------------------------------------------------
 .../apache/reef/runtime/common/files/UnixJVMPathProvider.java | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/reef/blob/17840f04/lang/java/reef-common/src/main/java/org/apache/reef/runtime/common/files/UnixJVMPathProvider.java
----------------------------------------------------------------------
diff --git 
a/lang/java/reef-common/src/main/java/org/apache/reef/runtime/common/files/UnixJVMPathProvider.java
 
b/lang/java/reef-common/src/main/java/org/apache/reef/runtime/common/files/UnixJVMPathProvider.java
index 9be2701..c7bc763 100644
--- 
a/lang/java/reef-common/src/main/java/org/apache/reef/runtime/common/files/UnixJVMPathProvider.java
+++ 
b/lang/java/reef-common/src/main/java/org/apache/reef/runtime/common/files/UnixJVMPathProvider.java
@@ -30,6 +30,11 @@ public final class UnixJVMPathProvider implements 
RuntimePathProvider {
 
   @Override
   public String getPath() {
-    return System.getenv("JAVA_HOME") + "/bin/" + "java";
+    return "{{JAVA_HOME}}/bin/java";
+  }
+
+  @Override
+  public String toString() {
+    return getPath();
   }
 }

Reply via email to