abstractdog commented on code in PR #98:
URL: https://github.com/apache/tez/pull/98#discussion_r1003235694


##########
tez-api/src/main/java/org/apache/tez/dag/api/TezConfiguration.java:
##########
@@ -260,6 +260,48 @@ public TezConfiguration(boolean loadDefaults) {
   public static final String TEZ_TASK_LOG_LEVEL = TEZ_TASK_PREFIX + 
"log.level";
   public static final String TEZ_TASK_LOG_LEVEL_DEFAULT = "INFO";
 
+  /**
+   * By this option, user can easily override the logging pattern which is 
applied in
+   * TezContainerLogAppender in AM, regardless of the environmental settings.
+   */
+  @ConfigurationScope(Scope.AM)
+  @ConfigurationProperty
+  public static final String TEZ_LOG_PATTERN_LAYOUT_AM = TEZ_AM_PREFIX + 
"log.pattern.layout";
+
+  /**
+   * By this option, user can easily override the logging pattern which is 
applied in
+   * TezContainerLogAppender in tasks, regardless of the environmental 
settings.
+   */
+  @ConfigurationScope(Scope.VERTEX)
+  @ConfigurationProperty
+  public static final String TEZ_LOG_PATTERN_LAYOUT_TASK = TEZ_TASK_PREFIX + 
"log.pattern.layout";

Review Comment:
   good catch, user should be able to turn this off by setting it to an empty 
string, I'm reflecting this in the configuration



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@tez.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to