Mingliang Liu created HADOOP-13470:
--------------------------------------

             Summary: GenericTestUtils$LogCapturer is flaky
                 Key: HADOOP-13470
                 URL: https://issues.apache.org/jira/browse/HADOOP-13470
             Project: Hadoop Common
          Issue Type: Bug
          Components: test, util
    Affects Versions: 2.8.0
            Reporter: Mingliang Liu
            Assignee: Mingliang Liu


{{GenericTestUtils$LogCapturer}} is useful for assertions against service logs. 
However it should be fixed in following aspects:
# In the constructor, it uses the stdout appender's layout.
{code}
Layout layout = Logger.getRootLogger().getAppender("stdout").getLayout();
{code}
However, the stdout appender may be named "console" or alike which makes the 
constructor throw NPE. Actually the layout does not matter and we can use a 
default pattern layout that only captures application logs.
# {{stopCapturing()}} method is not working. The major reason is that the 
{{appender}} internal variable is never assigned and thus removing it to stop 
capturing makes no sense.
# It does not support {{org.slf4j.Logger}} which is preferred to log4j in many 
modules.
# There is no unit test for it.

This jira is to address these.





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

---------------------------------------------------------------------
To unsubscribe, e-mail: common-dev-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-dev-h...@hadoop.apache.org

Reply via email to