This is an automated email from the ASF dual-hosted git repository.

kshukla pushed a commit to branch branch-0.9
in repository https://gitbox.apache.org/repos/asf/tez.git


The following commit(s) were added to refs/heads/branch-0.9 by this push:
     new 4c9580c  TEZ-4041. TestExtServicesWithLocalMode fails in docker 
(Jonathan Eagles via kshukla)
4c9580c is described below

commit 4c9580c8e2d6eac8d266e66bdcec941e18c86979
Author: Kuhu Shukla <[email protected]>
AuthorDate: Wed Feb 13 15:08:21 2019 -0600

    TEZ-4041. TestExtServicesWithLocalMode fails in docker (Jonathan Eagles via 
kshukla)
    
    (cherry picked from commit 62a1140be8bac21c4f18c3bf6f9cc124ea1a1bfb)
---
 .../tez/dag/app/taskcomm/TezTestServiceTaskCommunicatorImpl.java       | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git 
a/tez-ext-service-tests/src/test/java/org/apache/tez/dag/app/taskcomm/TezTestServiceTaskCommunicatorImpl.java
 
b/tez-ext-service-tests/src/test/java/org/apache/tez/dag/app/taskcomm/TezTestServiceTaskCommunicatorImpl.java
index 732c81a..377217b 100644
--- 
a/tez-ext-service-tests/src/test/java/org/apache/tez/dag/app/taskcomm/TezTestServiceTaskCommunicatorImpl.java
+++ 
b/tez-ext-service-tests/src/test/java/org/apache/tez/dag/app/taskcomm/TezTestServiceTaskCommunicatorImpl.java
@@ -61,8 +61,7 @@ public class TezTestServiceTaskCommunicatorImpl extends 
TezTaskCommunicatorImpl
 
     SubmitWorkRequestProto.Builder baseBuilder = 
SubmitWorkRequestProto.newBuilder();
 
-    // TODO Avoid reading this from the environment
-    
baseBuilder.setUser(System.getenv(ApplicationConstants.Environment.USER.name()));
+    baseBuilder.setUser(System.getProperty("user.name"));
     baseBuilder.setApplicationIdString(
         
taskCommunicatorContext.getApplicationAttemptId().getApplicationId().toString());
     baseBuilder

Reply via email to