Mailing-List: contact dev-help@struts.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Struts Developers List" <dev@struts.apache.org>
Received-SPF: pass (athena.apache.org: domain of musachy@gmail.com designates
 209.85.212.178 as permitted sender)
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:content-transfer-encoding;
        b=aCAaiPsgJB28Qbe6/mUMSlVXuviAcARkDhOXbCmde9aQki5FRFPMoyTsr1tObayrSF
         azgwYmliK2pFdfh1Rwnc3rT3ws3w+2364pRyww6d2jxz3l+Km4TYPsp47+/EhA+Kzyvq
         02DLAvpxd5nhSWDxN70gbyIbvjA0AGkkPBI/k=
MIME-Version: 1.0
In-Reply-To: <4A5B4054.4090004@newfield.org>
References: <d8bdeb6f0907111852o4c545f49u7ea67e5dab8710e4@mail.gmail.com>
	 <4A594774.3090807@newfield.org>
	 <d8bdeb6f0907111925u44bd1916lb8c7ca046507520@mail.gmail.com>
	 <d8bdeb6f0907120010t4012ee70h62404f95c0c41d0@mail.gmail.com>
	 <d8bdeb6f0907121131w69c795a4lae3812e3f3155532@mail.gmail.com>
	 <4A5AB88C.9040502@newfield.org>
	 <d8bdeb6f0907122212g63e4c1f5x9122e479d5b69ad7@mail.gmail.com>
	 <4A5B4054.4090004@newfield.org>
Date: Mon, 13 Jul 2009 09:29:57 -0700
Message-ID: <d8bdeb6f0907130929p10b3d532k55207917726caaab@mail.gmail.com>
Subject: Re: Class reloading
From: Musachy Barroso <musachy@gmail.com>
To: Struts Developers List <dev@struts.apache.org>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

I am thinking about adding another constant which holds a list of
regex, which if set, will limit what classes can be loaded by the
reloading classloader. The reason for this, is that I would like to
limit the reloaded classes to just my actions. If the reloading class
loader loads something that is not an action, there is a good chance
that instance of that class will be casted to the same class loaded by
the original class loader, which fails with a class cast exception.

musachy

On Mon, Jul 13, 2009 at 7:10 AM, Dale Newfield<dale@newfield.org> wrote:
> Musachy Barroso wrote:
>>
>> I could test the path as an absolute path, if it exist, use it,
>> otherwise try it as a relative path to context root, sounds good?
>
> Almost. =A0The File interface can tell you whether or not a given instanc=
e
> (path) isAbsolute(). =A0Clearly if the (absolute or relative) file does n=
ot
> exist it should not be loaded, but existence shouldn't help you decide wh=
ich
> way to interpret the path.
>
> This does bring up the question: =A0If the given (.class or .jar) file do=
es
> not exist at deployment time, will creation of it/changes to it later
> trigger reloading? =A0Either way, just to avoid confusion that bit of inf=
o
> should be added to the documentation. =A0(Hrm--looking at
> FilesystemAlterationListener and ClassReloadingXMLWebApplicationContext
> suggests that if the file exists at deployment time we should notice in
> onFileChange as you currently do, but if it did not exist (and we want to
> support later adding of classes/jars) we could be paying attention to
> onFileCreate/onDirectoryCreate events, too.)
>
> Oh, and once again: =A0Musachy++
>
> Lately it seems like Struts is improving more due to the heroic effort of=
 a
> few individuals than as a result of a community effort. =A0(I have a feel=
ing
> that this has fairly consistently been the case throughout the lifetime o=
f
> struts v1 and v2.) =A0I'm happy that struts is improving, so I'm not tryi=
ng to
> discourage heroic effort, but I'm wondering if anyone has any suggestions
> for how to get more involvement?
>
> -Dale
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
> For additional commands, e-mail: dev-help@struts.apache.org
>
>


--=20
"Hey you! Would you help me to carry the stone?" Pink Floyd

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org