Return-Path: X-Original-To: apmail-commons-dev-archive@www.apache.org Delivered-To: apmail-commons-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 543DFD85F for ; Fri, 4 Jan 2013 16:48:26 +0000 (UTC) Received: (qmail 58881 invoked by uid 500); 4 Jan 2013 16:48:25 -0000 Delivered-To: apmail-commons-dev-archive@commons.apache.org Received: (qmail 58785 invoked by uid 500); 4 Jan 2013 16:48:25 -0000 Mailing-List: contact dev-help@commons.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: "Commons Developers List" Delivered-To: mailing list dev@commons.apache.org Received: (qmail 58769 invoked by uid 99); 4 Jan 2013 16:48:25 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 04 Jan 2013 16:48:25 +0000 X-ASF-Spam-Status: No, hits=-0.0 required=5.0 tests=RCVD_IN_DNSWL_NONE,SPF_HELO_PASS,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of jak-commons-dev@m.gmane.org designates 80.91.229.3 as permitted sender) Received: from [80.91.229.3] (HELO plane.gmane.org) (80.91.229.3) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 04 Jan 2013 16:48:17 +0000 Received: from list by plane.gmane.org with local (Exim 4.69) (envelope-from ) id 1TrARC-0008E0-RY for dev@commons.apache.org; Fri, 04 Jan 2013 17:48:06 +0100 Received: from hsi-kbw-095-208-058-089.hsi5.kabel-badenwuerttemberg.de ([95.208.58.89]) by main.gmane.org with esmtp (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Fri, 04 Jan 2013 17:48:06 +0100 Received: from joerg.schaible by hsi-kbw-095-208-058-089.hsi5.kabel-badenwuerttemberg.de with local (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Fri, 04 Jan 2013 17:48:06 +0100 X-Injected-Via-Gmane: http://gmane.org/ To: dev@commons.apache.org From: =?UTF-8?B?SsO2cmc=?= Schaible Subject: Re: [configuration] Design discussion Date: Fri, 04 Jan 2013 17:47:40 +0100 Lines: 40 Message-ID: References: <50E3242A.5030301@oliver-heger.de> Reply-To: joerg.schaible@gmx.de Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8Bit X-Complaints-To: usenet@ger.gmane.org X-Gmane-NNTP-Posting-Host: hsi-kbw-095-208-058-089.hsi5.kabel-badenwuerttemberg.de User-Agent: KNode/4.9.3 X-Virus-Checked: Checked by ClamAV on apache.org Hi Oliver, Oliver Heger wrote: > Hi, > > recently I have worked on code regarding the creation of Configuration > objects and reloading support. I have created two Jira tickets [1, 2] > with a description of the problems I see in the current design. > > The code in SVN (mainly in the new builder package) should be sufficient > to get a good impression about the direction I would like to go. It is > far more than a pure proof of concept. > > However, following this road means some significant changes in the > design and in the way client code uses our classes. Therefore, I would > really appreciate feedback from other committers also interested in this > component. > > My main question is: Can we replace the reloading mechanisms available > in 1.x by an approach based on builders as currently implemented in > trunk (e.g. classes > o.a.c.c.builder.ReloadingFileBasedConfigurationBuilder, > o.a.c.c.reloading.ReloadingController)? If the answer is 'yes', we could > go forward and significantly simplify most of the file-based > configuration implementations. > > Thanks > Oliver > > [1] https://issues.apache.org/jira/browse/CONFIGURATION-519 > [2] https://issues.apache.org/jira/browse/CONFIGURATION-520 simply go-on with these changes. IMHO it looks good and since separation of concern leads here to significant code simplification, it's for the best of us devs and also for our users in the long term. Especially since the new approach brings also improved functionality. Cheers, Jörg --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org For additional commands, e-mail: dev-help@commons.apache.org