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

Missing documentation for Max index limit in DefaultRolloverStrategy

    Details

    • Type: Documentation
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.9.0
    • Component/s: Documentation, Plugins
    • Labels:
      None
    • Flags:
      Important

      Description

      Guys, this is regarding the missing documentation of the max index limit to the DefaultRolloverStrategy. By default, this is 7 based on the implementation. Unfortunately, this is not documented.

      Because of the lack of the documentation, one might assume it to be an unbounded limit (or rather high?). This might lead to data-loss, which happened in my case. Please document it.

      The original discussion is here: https://lists.apache.org/thread.html/092122655c1d0fba65712dc82f17a6f619a7e0a716a6f4130272773e@%3Clog4j-user.logging.apache.org%3E

        Issue Links

          Activity

          Hide
          githubbot ASF GitHub Bot added a comment -

          GitHub user tckb opened a pull request:

          https://github.com/apache/logging-log4j2/pull/72

          (doc) LOG4J2-1877: updated DefaultRolloverStrategy documentation

          hey, I've updated the site documentation to include the default max value of the defaultrolloverstrategy.

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

          $ git pull https://github.com/tckb/logging-log4j2 master

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

          https://github.com/apache/logging-log4j2/pull/72.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 #72


          commit ca4e9f51e21a67c26142d02c7b8259e62f48f3b8
          Author: tckb <tckb.504@gmail.com>
          Date: 2017-04-19T05:07:25Z

          (doc) LOG4J2-1877: updated 'DefaultRolloverStrategy' documentation to include the default value of 'max' attribute


          Show
          githubbot ASF GitHub Bot added a comment - GitHub user tckb opened a pull request: https://github.com/apache/logging-log4j2/pull/72 (doc) LOG4J2-1877 : updated DefaultRolloverStrategy documentation hey, I've updated the site documentation to include the default max value of the defaultrolloverstrategy. You can merge this pull request into a Git repository by running: $ git pull https://github.com/tckb/logging-log4j2 master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/logging-log4j2/pull/72.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 #72 commit ca4e9f51e21a67c26142d02c7b8259e62f48f3b8 Author: tckb <tckb.504@gmail.com> Date: 2017-04-19T05:07:25Z (doc) LOG4J2-1877 : updated 'DefaultRolloverStrategy' documentation to include the default value of 'max' attribute
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 4ca30b52bcfdbd55cbb96061c2d8d7a5c2d90dd4 in logging-log4j2's branch refs/heads/master from Gary Gregory
          [ https://git-wip-us.apache.org/repos/asf?p=logging-log4j2.git;h=4ca30b5 ]

          LOG4J2-1877
          Missing documentation for Max index limit in DefaultRolloverStrategy.

          Show
          jira-bot ASF subversion and git services added a comment - Commit 4ca30b52bcfdbd55cbb96061c2d8d7a5c2d90dd4 in logging-log4j2's branch refs/heads/master from Gary Gregory [ https://git-wip-us.apache.org/repos/asf?p=logging-log4j2.git;h=4ca30b5 ] LOG4J2-1877 Missing documentation for Max index limit in DefaultRolloverStrategy.
          Hide
          garydgregory Gary Gregory added a comment -

          In git master; please verify and close. Closes #72.

          Show
          garydgregory Gary Gregory added a comment - In git master; please verify and close. Closes #72.
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 4ca30b52bcfdbd55cbb96061c2d8d7a5c2d90dd4 in logging-log4j2's branch refs/heads/java9NoMultiRelease from Gary Gregory
          [ https://git-wip-us.apache.org/repos/asf?p=logging-log4j2.git;h=4ca30b5 ]

          LOG4J2-1877
          Missing documentation for Max index limit in DefaultRolloverStrategy.

          Show
          jira-bot ASF subversion and git services added a comment - Commit 4ca30b52bcfdbd55cbb96061c2d8d7a5c2d90dd4 in logging-log4j2's branch refs/heads/java9NoMultiRelease from Gary Gregory [ https://git-wip-us.apache.org/repos/asf?p=logging-log4j2.git;h=4ca30b5 ] LOG4J2-1877 Missing documentation for Max index limit in DefaultRolloverStrategy.
          Hide
          jvz Matt Sicker added a comment -

          Gary, you merged this without the author info again.

          Show
          jvz Matt Sicker added a comment - Gary, you merged this without the author info again.
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit ca4e9f51e21a67c26142d02c7b8259e62f48f3b8 in logging-log4j2's branch refs/heads/master from tckb
          [ https://git-wip-us.apache.org/repos/asf?p=logging-log4j2.git;h=ca4e9f5 ]

          (doc) LOG4J2-1877: updated 'DefaultRolloverStrategy' documentation to include the default value of 'max' attribute

          Show
          jira-bot ASF subversion and git services added a comment - Commit ca4e9f51e21a67c26142d02c7b8259e62f48f3b8 in logging-log4j2's branch refs/heads/master from tckb [ https://git-wip-us.apache.org/repos/asf?p=logging-log4j2.git;h=ca4e9f5 ] (doc) LOG4J2-1877 : updated 'DefaultRolloverStrategy' documentation to include the default value of 'max' attribute
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit d0d331fb93ec3645770b67454c4c11e5706dfb61 in logging-log4j2's branch refs/heads/master from Matt Sicker
          [ https://git-wip-us.apache.org/repos/asf?p=logging-log4j2.git;h=d0d331f ]

          Merge commit 'refs/pull/72/head' of https://github.com/apache/logging-log4j2

          This closes #72 and is related to LOG4J2-1877.

          Show
          jira-bot ASF subversion and git services added a comment - Commit d0d331fb93ec3645770b67454c4c11e5706dfb61 in logging-log4j2's branch refs/heads/master from Matt Sicker [ https://git-wip-us.apache.org/repos/asf?p=logging-log4j2.git;h=d0d331f ] Merge commit 'refs/pull/72/head' of https://github.com/apache/logging-log4j2 This closes #72 and is related to LOG4J2-1877 .
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user asfgit closed the pull request at:

          https://github.com/apache/logging-log4j2/pull/72

          Show
          githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/logging-log4j2/pull/72

            People

            • Assignee:
              garydgregory Gary Gregory
              Reporter:
              _chandra chandra
            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development