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

ASF GitHub Bot commented on TWILL-222:
--------------------------------------

GitHub user yaojiefeng opened a pull request:

    https://github.com/apache/twill/pull/37

    (Twill-222) fix for default root log level

    JIRA: https://issues.apache.org/jira/browse/TWILL-222
    
    We should not assume the root log level as INFO in TwillPreparer. Removed 
the INFO level from TwillPreparer to not override what is in the logback.xml.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/yaojiefeng/twill 
fix/Twill-222-fix-twillPreparer-root-loglevel

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/twill/pull/37.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #37
    
----
commit 33cf94d661f854b5770ec448531331cf2faafaf7
Author: yaojiefeng <yao...@cask.co>
Date:   2017-03-16T23:57:05Z

    (Twill-222) fix for default root log level

----


> TwillRunnable will always have the ROOT log level INFO if we do not 
> explicitly set the log level in TwillPreparer.
> ------------------------------------------------------------------------------------------------------------------
>
>                 Key: TWILL-222
>                 URL: https://issues.apache.org/jira/browse/TWILL-222
>             Project: Apache Twill
>          Issue Type: Bug
>            Reporter: Yaojie Feng
>            Assignee: Yaojie Feng
>
> TwillPreparer now blindly assumes the ROOT logger has INFO log level if we do 
> not set it no matter what it actually is in the logback.xml. And after we 
> configure the logback.xml in ServiceMain, the root log level get overrided to 
> INFO.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to