Uploaded image for project: 'Log4j 2'
  1. Log4j 2
  2. LOG4J2-1620

2.7-rc1: RollingFileAppender immediateFlush default value should be true, not false

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 2.7
    • None
    • Appenders
    • None

    Description

      The default value of the immediaFlush property in RollingFile appender was changed from true to false (compared with 2.6.2). The change was introduced with

      a82794f06e3bc095ef0bca41b98c5d1ea915de76 is the first bad commit
      commit a82794f06e3bc095ef0bca41b98c5d1ea915de76
      Author: Gary Gregory <ggregory@apache.org>
      Date:   Fri Aug 19 17:27:26 2016 -0700
      
          [LOG4J2-1504] RollingFileAppender should be able to create files
          on-demand.
      
      :040000 040000 3d3faa81429c2ff83db3b1672a2700e2efe770e3 279da0b08553454f69fdbebfcdb855b21dc7c2ae M	log4j-core
      :040000 040000 9548cb83e88b752d42e80ef6cab3463eed9b868c 5a23c7038b1f4bb49c8b6b8727d4a681db3c2ae8 M	src
      

      It looks to me from the commit message that this change was not intended. And it breaks my current usage. I'd have to explicitly set the value to true now in order to avoid a regression in my code.

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            ggregory Gary D. Gregory
            zlosch Sascha Scholz
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment