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

Sergey Shelukhin edited comment on TEZ-2917 at 10/30/15 10:45 PM:
------------------------------------------------------------------

MRInputLegacy deferring initialization - how is this useful. Another thing btw 
is that whatever developer can figure out by just looking at config file, 
metadata, etc., and code should not be on info level. I.e. logging configs and 
other such things.
getContext().getDestinationVertexName() + ": "
         + "outputFormat=" + outputFormatClassName
         + ", using newmapreduce API=" + useNewApi - that looks like it could 
just be figured out from the job and config

Using oldApi, MRpartitionerClass= - same

Waiting for N... - is just logged many times all the time and it seems 
extremely situational, when some small piece of code got stuck. Start of the 
process is already logged, and so is the end (I can restore these back to info) 
so it doesn't give much information even if it does hang; in 99.99% cases it's 
just noise.

Cleaning up task, Initializing task are extremely situational given the 
surrounding log lines (i.e. running... is logged at info).

I dunno, I don't really care either way, this is based on users' complaints 
that too much obscure stuff that doesn't matter is getting logged. [~hagleitn] 
any opinion?


was (Author: sershe):
MRInputLegacy deferring initialization - how is this useful. Another thing btw 
is that whatever developer can figure out by just looking at config file, 
metadata, etc., and code should not be on info level. I.e. logging configs and 
other such things.
getContext().getDestinationVertexName() + ": "
         + "outputFormat=" + outputFormatClassName
         + ", using newmapreduce API=" + useNewApi - that looks like it could 
just be figured out from the job and config

Using oldApi, MRpartitionerClass= - same

Waiting for N... - is just logged many times all the time and it seems 
extremely situational, when some small piece of code got stuck. Maybe it can 
log waiting once.

Cleaning up task, Initializing task are extremely situational given the 
surrounding log lines (i.e. running... is logged at info).

I dunno, I don't really care either way, this is based on users' complaints 
that too much obscure stuff that doesn't matter is getting logged. [~hagleitn] 
any opinion?

> change some logs from info to debug
> -----------------------------------
>
>                 Key: TEZ-2917
>                 URL: https://issues.apache.org/jira/browse/TEZ-2917
>             Project: Apache Tez
>          Issue Type: Bug
>            Reporter: Sergey Shelukhin
>            Assignee: Sergey Shelukhin
>         Attachments: TEZ-2917.patch
>
>
> I've done a highly unscientific summarization of the logs from some random 
> queries, and will now change some log statements that are the most prevalent 
> and not extremely useful from info to debug.



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

Reply via email to