[jira] [Updated] (LOG4J2-277) Wasted work in RollingAppenderSizeTest.testAppender()

["Adrian Nistor (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/LOG4J2-277?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Adrian Nistor updated LOG4J2-277:
---------------------------------

    Attachment: patchAll.diff
                patch6.diff
                patch5.diff
                patch4.diff
                patch3.diff
                patch2.diff
                patch.diff
    
> Wasted work in RollingAppenderSizeTest.testAppender()
> -----------------------------------------------------
>
>                 Key: LOG4J2-277
>                 URL: https://issues.apache.org/jira/browse/LOG4J2-277
>             Project: Log4j 2
>          Issue Type: Bug
>    Affects Versions: 2.0-beta7
>         Environment: any
>            Reporter: Adrian Nistor
>            Priority: Minor
>              Labels: patch, performance
>         Attachments: patch2.diff, patch3.diff, patch4.diff, patch5.diff, patch6.diff, patchAll.diff, patch.diff
>
>
> The problem appears in Log4j 2.0-beta7 and in revision 1491481.  I
> attached a one-line patch (patch.diff) that fixes it.  This problem
> and the attached patch are similar to the previously fixed LOG4J2-273
> and LOG4J2-274, and their patches.
> In method "RollingAppenderSizeTest.testAppender", the loop over
> "files" should break immediately after "found" is set to "true".  All
> the iterations after "found" is set to "true" do not perform any
> useful work, at best they just set "found" again to "true".
> Similar problems exist in
> "RollingAppenderTimeAndSizeTest.testAppender",
> "RollingAppenderTimeTest.testAppender",
> "RollingAppenderUncompressedTest.testAppender", "PerfTest.doMain", and
> "AdvertiserTest.verifyExpectedEntriesAdvertised".  I attached patches
> patch2.diff, patch3.diff, patch4.diff, patch5.diff, and patch6.diff
> for these methods.  I also attached a patch (patchAll.diff) that
> aggregates all these changes in one single patch.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org