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

Prasanth Jayachandran commented on HIVE-13262:
----------------------------------------------

[~sershe] This patch adds logger with names for different logical pieces in 
LlapIo. Also removes some unused methods and classes. I have added 
parameterized logging all over and guarded some places with isXXXEnabled where 
I thought the log line will be expensive. I am not sure if some lines should go 
to info/debug/trace level. Can you please comment on RB if you think some lines 
should be logged at different level instead?

> LLAP: Remove log levels from DebugUtils
> ---------------------------------------
>
>                 Key: HIVE-13262
>                 URL: https://issues.apache.org/jira/browse/HIVE-13262
>             Project: Hive
>          Issue Type: Bug
>    Affects Versions: 2.1.0
>            Reporter: Prasanth Jayachandran
>            Assignee: Prasanth Jayachandran
>         Attachments: HIVE-13262.1.patch
>
>
> DebugUtils has many hardcoded log levels. To enable logging we need to 
> recompile code with desired value. Instead configure add loggers for these 
> classes with log levels via log4j properties. Also use parametrized logging 
> in IO elevator. 



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

Reply via email to