Re: Max index limit in DefaultRolloverStrategy

2017-04-18 Thread Ralph Goers
FYI - the documentation for the web site is in the project at src/site/xdoc/manual. You can create a patch for that the same way you would patch any of the java source. Ralph > On Apr 18, 2017, at 10:20 AM, Chandra > wrote: > > Hi Gary, > > I’d love

Re: Max index limit in DefaultRolloverStrategy

2017-04-18 Thread Chandra
Hi Gary, I’d love to, but this ticket is primarily to change the documentation and it’d have a minor amendments in the document on the website. I’m not exactly sure how to send a patch to this :-/ 1) https://logging.apache.org/log4j/2.x/manual/appenders.html#DefaultRolloverStrategy

Re: Max index limit in DefaultRolloverStrategy

2017-04-18 Thread Gary Gregory
Patches welcome ;-) Gary On Apr 18, 2017 1:18 AM, "Chandra" wrote: Hi guys, any update on this ticket? https://issues.apache.org/jira/browse/LOG4J2-1877 thanks, Chandra On 12 Apr 2017, 12:14 AM +0530, Chandra ,

Re: Max index limit in DefaultRolloverStrategy

2017-04-18 Thread Chandra
Hi guys, any update on this ticket? https://issues.apache.org/jira/browse/LOG4J2-1877 thanks, Chandra On 12 Apr 2017, 12:14 AM +0530, Chandra , wrote: > Yes, that’d be great.  7 seems to too low for frequently filled logs. > And also guys, please document

Re: Max index limit in DefaultRolloverStrategy

2017-04-11 Thread Chandra
Yes, that’d be great.  7 seems to too low for frequently filled logs. And also guys, please document it. I am raising a JIRA ticket in this regards ( thanks Remko for the reminder). Also yes, I have set a default purge policy, this is 90 days in our use case. Best, Chandra On 12 Apr 2017, 12:07

Re: Max index limit in DefaultRolloverStrategy

2017-04-11 Thread Ralph Goers
I believe starting with 2.8.1 we no longer scan the directory so it is much faster. We could probably raise the default limit. Ralph > On Apr 11, 2017, at 11:12 AM, Remko Popma wrote: > > Sorry to hear that! > > You are right that the docs currently don't mention the

Re: Max index limit in DefaultRolloverStrategy

2017-04-11 Thread Remko Popma
Sorry to hear that! You are right that the docs currently don't mention the default value for the max attribute. We should fix the docs. Can I ask you to raise a Jira for this? As for why not a large default value, there is a performance impact: all files within the range need to be renamed

Max index limit in DefaultRolloverStrategy

2017-04-09 Thread Chandra
Hi guys, Something I noticed in my production, the maximum limit of index (i.e., the max #files to be rolled before an overwrite occurs) is 7 by default. This is not mentioned anywhere in any of the documentation. I was only able to confirm this by actually looking into the source code [1].