[ 
https://issues.apache.org/jira/browse/TEZ-3326?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15450466#comment-15450466
 ] 

Hitesh Shah commented on TEZ-3326:
----------------------------------

bq. It appears that the motivation for returning a string as opposed to 
directly logging is to avoid logging a line with no message.

I guess the function itself could have done the null check but we can keep the 
code as is for now.

bq. Is there a way I can do that with conf.getStrings()?

Not in a one-liner but how about using conf.getTrimmedStringCollection() and if 
empty, falling back to a default where the default is a collection of strings 
which does not require additional parsing?

> Display JVM system properties in AM and task logs
> -------------------------------------------------
>
>                 Key: TEZ-3326
>                 URL: https://issues.apache.org/jira/browse/TEZ-3326
>             Project: Apache Tez
>          Issue Type: Improvement
>            Reporter: Ming Ma
>            Assignee: Eric Badger
>         Attachments: TEZ-3326.001.patch, TEZ-3326.002.patch, 
> TEZ-3326.003.patch
>
>
> MapReduce displays JVM system properties via config 
> {{mapreduce.jvm.system-properties-to-log}} in both AM and task log . This is 
> useful to debug env setting such as java version, etc. It is useful to have 
> such logging in Tez.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to