You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tez.apache.org by je...@apache.org on 2015/02/12 00:41:15 UTC

tez git commit: TEZ-2073. SimpleHistoryLoggingService cannot be read by log aggregation (umask) (jeagles)

Repository: tez
Updated Branches:
  refs/heads/master 595330638 -> 53604b5b8


TEZ-2073. SimpleHistoryLoggingService cannot be read by log aggregation (umask) (jeagles)


Project: http://git-wip-us.apache.org/repos/asf/tez/repo
Commit: http://git-wip-us.apache.org/repos/asf/tez/commit/53604b5b
Tree: http://git-wip-us.apache.org/repos/asf/tez/tree/53604b5b
Diff: http://git-wip-us.apache.org/repos/asf/tez/diff/53604b5b

Branch: refs/heads/master
Commit: 53604b5b80bb3e70cf1fbeb5aabed0b5bcf854d3
Parents: 5953306
Author: Jonathan Eagles <je...@gmail.com>
Authored: Wed Feb 11 17:41:02 2015 -0600
Committer: Jonathan Eagles <je...@gmail.com>
Committed: Wed Feb 11 17:41:02 2015 -0600

----------------------------------------------------------------------
 CHANGES.txt                                                       | 1 +
 .../tez/dag/history/logging/impl/SimpleHistoryLoggingService.java | 3 +--
 2 files changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tez/blob/53604b5b/CHANGES.txt
----------------------------------------------------------------------
diff --git a/CHANGES.txt b/CHANGES.txt
index bfecc37..1d86ba4 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -61,6 +61,7 @@ Release 0.6.1: Unreleased
 INCOMPATIBLE CHANGES
 
 ALL CHANGES:
+  TEZ-2073. SimpleHistoryLoggingService cannot be read by log aggregation (umask)
   TEZ-2078. Tez UI: Task logs url use in-progress url causing various errors.
   TEZ-2077. Tez UI: No diagnostics on Task Attempt Details page if task attempt failed. 
   TEZ-2065. Setting up tez.tez-ui.history-url.base with a trailing slash can result in failures to redirect correctly. 

http://git-wip-us.apache.org/repos/asf/tez/blob/53604b5b/tez-dag/src/main/java/org/apache/tez/dag/history/logging/impl/SimpleHistoryLoggingService.java
----------------------------------------------------------------------
diff --git a/tez-dag/src/main/java/org/apache/tez/dag/history/logging/impl/SimpleHistoryLoggingService.java b/tez-dag/src/main/java/org/apache/tez/dag/history/logging/impl/SimpleHistoryLoggingService.java
index 95ee010..145c0e9 100644
--- a/tez-dag/src/main/java/org/apache/tez/dag/history/logging/impl/SimpleHistoryLoggingService.java
+++ b/tez-dag/src/main/java/org/apache/tez/dag/history/logging/impl/SimpleHistoryLoggingService.java
@@ -29,7 +29,6 @@ import org.apache.hadoop.conf.Configuration;
 import org.apache.hadoop.fs.FSDataOutputStream;
 import org.apache.hadoop.fs.FileSystem;
 import org.apache.hadoop.fs.Path;
-import org.apache.tez.common.TezUtilsInternal;
 import org.apache.tez.dag.api.TezConfiguration;
 import org.apache.tez.dag.history.DAGHistoryEvent;
 import org.apache.tez.dag.history.logging.HistoryLoggingService;
@@ -67,7 +66,7 @@ public class SimpleHistoryLoggingService extends HistoryLoggingService {
       LOG.info("Log file location for SimpleHistoryLoggingService not specified, defaulting to"
           + " containerLogDir=" + logDir);
       Path p;
-      logFileFS = FileSystem.getLocal(conf);
+      logFileFS = FileSystem.getLocal(conf).getRawFileSystem();
       if (logDir != null) {
         p = new Path(logDir, logFileName);
       } else {