Return-Path: Delivered-To: apmail-commons-dev-archive@www.apache.org Received: (qmail 90620 invoked from network); 12 Oct 2010 11:33:33 -0000 Received: from unknown (HELO mail.apache.org) (140.211.11.3) by 140.211.11.9 with SMTP; 12 Oct 2010 11:33:33 -0000 Received: (qmail 34982 invoked by uid 500); 12 Oct 2010 11:33:33 -0000 Delivered-To: apmail-commons-dev-archive@commons.apache.org Received: (qmail 34846 invoked by uid 500); 12 Oct 2010 11:33:32 -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 34838 invoked by uid 99); 12 Oct 2010 11:33:32 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 12 Oct 2010 11:33:32 +0000 X-ASF-Spam-Status: No, hits=0.0 required=10.0 tests=FREEMAIL_FROM,RCVD_IN_DNSWL_NONE,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (nike.apache.org: domain of simone.tripodi@gmail.com designates 74.125.82.171 as permitted sender) Received: from [74.125.82.171] (HELO mail-wy0-f171.google.com) (74.125.82.171) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 12 Oct 2010 11:33:25 +0000 Received: by wya21 with SMTP id 21so2094919wya.30 for ; Tue, 12 Oct 2010 04:33:05 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:received:received:in-reply-to :references:date:message-id:subject:from:to:content-type; bh=+zfbQbXUGxvNTP7WOYapjdIbX7qHe2j/pNlhjJz8+cI=; b=mJU2hSrleuGy4ns0/TI16iCp+XfD81OdEGBMw6EI9Jasss2Sp8ubDqdvjgkXA69sja LdTgIursf7PJyqvf51HBWJo45xC25BcpStVYxQc/Oj7ywZtw+Mr7jRSO+bQdNtkZr4kJ MAk8KCBTRVnKXn3gwcGlack9xnhVumssMt0XY= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; b=ZpuvLj6/4p0M8l4nQnXZ85BS4epqvgddZd+KhHyNf2/jysqCBkouIZRSZ17kPCmlNu 8kWlUJfwUue0Ax+W2a3dY/iEajiGDsESEtRi5HjzhWLQ3znC4mzKQKtbjWyeY9ZxD3gf kw0YavpZ9JKGWHXT1+pYA0B8yZ6gUm0wF9cXc= MIME-Version: 1.0 Received: by 10.227.157.200 with SMTP id c8mr6972345wbx.69.1286883163872; Tue, 12 Oct 2010 04:32:43 -0700 (PDT) Received: by 10.216.36.195 with HTTP; Tue, 12 Oct 2010 04:32:43 -0700 (PDT) In-Reply-To: References: Date: Tue, 12 Oct 2010 13:32:43 +0200 Message-ID: Subject: Re: [pool] runtime re-configuration From: Simone Tripodi To: Commons Developers List Content-Type: text/plain; charset=UTF-8 X-Virus-Checked: Checked by ClamAV on apache.org Hi Seb, I totally agree, I'm for this solution, BTW I'll wait the Phil's opinion that knows more than me. Thanks! Simo http://people.apache.org/~simonetripodi/ http://www.99soft.org/ On Tue, Oct 12, 2010 at 12:50 PM, sebb wrote: > On 12 October 2010 10:20, Simone Tripodi wrote: >> Hi all guys, >> while fixing the deprecated properties in classes like >> StackKeyedObjectPool[1], I noticed this class instance was >> re-configured during the test[2] (see line 126); is the >> "reconfigure-in-runtime" a pool feature we want? I'm asking because >> I've never experienced the pool reconfiguration (I've never had the >> need to do it) so I honestly don't know which is the wished behavior. >> In the scenario we want to keep this feature, since I'm converting >> fields as private, I need to add setters. >> Just let me know!!! Have a nice day, > > AFAIK, the tests that modify the configuration are to be dropped once > the variables are made private. > The idea was not just to make the variables private, but to make them > final as far as possible to improve thread safety. > > Perhaps Phil can confirm this? > >> Simo >> >> [1] http://s.apache.org/bjw >> [2] http://s.apache.org/qB >> >> http://people.apache.org/~simonetripodi/ >> http://www.99soft.org/ >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org >> For additional commands, e-mail: dev-help@commons.apache.org >> >> > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org > For additional commands, e-mail: dev-help@commons.apache.org > > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org For additional commands, e-mail: dev-help@commons.apache.org