Return-Path: X-Original-To: apmail-logging-log4net-dev-archive@www.apache.org Delivered-To: apmail-logging-log4net-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 02DF37499 for ; Mon, 12 Sep 2011 09:44:09 +0000 (UTC) Received: (qmail 11274 invoked by uid 500); 12 Sep 2011 09:44:06 -0000 Delivered-To: apmail-logging-log4net-dev-archive@logging.apache.org Received: (qmail 8068 invoked by uid 500); 12 Sep 2011 09:43:40 -0000 Mailing-List: contact log4net-dev-help@logging.apache.org; run by ezmlm Precedence: bulk list-help: list-unsubscribe: List-Post: Reply-To: "Log4NET Dev" List-Id: Delivered-To: mailing list log4net-dev@logging.apache.org Received: (qmail 8028 invoked by uid 99); 12 Sep 2011 09:43:33 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 12 Sep 2011 09:43:33 +0000 X-ASF-Spam-Status: No, hits=0.7 required=5.0 tests=SPF_NEUTRAL X-Spam-Check-By: apache.org Received-SPF: neutral (athena.apache.org: local policy) Received: from [88.84.128.168] (HELO samaflost.de) (88.84.128.168) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 12 Sep 2011 09:43:26 +0000 Received: from localhost (localhost.localdomain [127.0.0.1]) by samaflost.de (Postfix) with ESMTP id 53E9940F00D5 for ; Mon, 12 Sep 2011 11:43:05 +0200 (CEST) Received: from samaflost.de ([127.0.0.1]) by localhost (v35516.1blu.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id jDnxn5CAezP3 for ; Mon, 12 Sep 2011 11:43:05 +0200 (CEST) Received: by samaflost.de (Postfix, from userid 1000) id 0AD4E40F00D4; Mon, 12 Sep 2011 11:43:05 +0200 (CEST) From: Stefan Bodewig To: log4net-dev@logging.apache.org Subject: Re: The state of RollingFileAppender References: Date: Mon, 12 Sep 2011 11:43:05 +0200 In-Reply-To: (Roy Chastain's message of "Sun, 11 Sep 2011 21:54:53 -0400") Message-ID: <877h5e6qqe.fsf@v35516.1blu.de> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.3 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii On 2011-09-12, Roy Chastain wrote: > When I looked at this code a few years ago, I thought it was overly > complicated and obtuse. Since spending the day with it today, and > discovering the invalid assumption, I stand by my original opinion. I was afraid you'd say that when you volunteered to look into it. So far I stayed away from RFA and concentrated on the other issues - and never had any reason to read into RFA's code in the past. We seem to belong to the lucky few who haven't faced any of its internal issues. But just from reading through the sevaral issues raised against it it was clear to me that it must contain a multitude of problems that have historically piled up. > After looking at the 14 or so bug reports against RFA and remembering a > few that I never submitted, I think RFA needs a major simplification via > a rewrite to better handle gaps in the file names/numbers etc. OK. I think it would be good to push all of the RFA issues from 1.2.11 to 1.2 MAINTENANCE then and live with the fact that there will be known issues with it in 1.2.11 - or do you expect to have a drop-in replacement ready in a time frame of - say - a few weeks? Unfortunately I still don't have a build environment for the "older" frameworks (but am far from having given up on it) and this seems to be the major hurdle for the 1.2.11 release right now. > I am open to suggestions as to what features to add/delete from a > rewritten RFA. I know many people, including myself, have wanted a max > number of files per time increment when rolled by size and time. I think there is a JIRA issue (with patch IIRC) to that effect. Stefan