You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tuscany.apache.org by Luciano Resende <lu...@gmail.com> on 2008/09/30 04:38:48 UTC
Inverse classloading versus Hidden class, was Re: svn commit: r700138 - /tuscany/java/sca/demos/alert-aggregator-webapp/src/main/webapp/WEB-INF/geronimo-web.xml
I think we should better understand what is causing this change in
behavior in Geronimo and not assume this change should be the concrete
fix.
It would be bad to require our users to create these hidden classes
list when deploying their web app in geronimo, it would require too
much understanding of the Tuscany internals, and SCA is all about
simplicity.
This thread in geronimo-dev [1] should give us more information on the subject.
[1] http://www.mail-archive.com/dev@geronimo.apache.org/msg65191.html
On Mon, Sep 29, 2008 at 7:29 AM, <an...@apache.org> wrote:
> Author: antelder
> Date: Mon Sep 29 07:29:42 2008
> New Revision: 700138
>
> URL: http://svn.apache.org/viewvc?rev=700138&view=rev
> Log:
> TUSCANY-2622: Apply patch from Kevan Miller to Fix geronimo runtime errors caused by inverse-classloading in alert-aggregator-webapp demo
>
> Modified:
> tuscany/java/sca/demos/alert-aggregator-webapp/src/main/webapp/WEB-INF/geronimo-web.xml
>
> Modified: tuscany/java/sca/demos/alert-aggregator-webapp/src/main/webapp/WEB-INF/geronimo-web.xml
> URL: http://svn.apache.org/viewvc/tuscany/java/sca/demos/alert-aggregator-webapp/src/main/webapp/WEB-INF/geronimo-web.xml?rev=700138&r1=700137&r2=700138&view=diff
> ==============================================================================
> --- tuscany/java/sca/demos/alert-aggregator-webapp/src/main/webapp/WEB-INF/geronimo-web.xml (original)
> +++ tuscany/java/sca/demos/alert-aggregator-webapp/src/main/webapp/WEB-INF/geronimo-web.xml Mon Sep 29 07:29:42 2008
> @@ -26,7 +26,12 @@
> <d:version>1.4-SNAPSHOT</d:version>
> <d:type>war</d:type>
> </d:moduleId>
> - <!-- TUSCANY-1986 -->
> - <d:inverse-classloading />
> + <!-- TUSCANY-2622 -->
> + <d:hidden-classes>
> + <d:filter>org.apache.axiom</d:filter>
> + <d:filter>org.apache.axis2</d:filter>
> + <d:filter>org.apache.commons</d:filter>
> + <d:filter>org.jdom</d:filter>
> + </d:hidden-classes>
> </d:environment>
> </web-app>
>
>
>
--
Luciano Resende
Apache Tuscany, Apache PhotArk
http://people.apache.org/~lresende
http://lresende.blogspot.com/
Re: Inverse classloading versus Hidden class, was Re: svn commit: r700138 - /tuscany/java/sca/demos/alert-aggregator-webapp/src/main/webapp/WEB-INF/geronimo-web.xml
Posted by Luciano Resende <lu...@gmail.com>.
Then, should we update all instances of the geronimo-web.xml files ?
On Mon, Sep 29, 2008 at 11:39 PM, ant elder <an...@gmail.com> wrote:
> I don't disagree with you, and with the ongoing SCA JEE work I expect there
> will be a more changes around this area. This fix gets things working for
> now though which I found useful.
>
> ...ant
>
> On Tue, Sep 30, 2008 at 3:38 AM, Luciano Resende <lu...@gmail.com>
> wrote:
>>
>> I think we should better understand what is causing this change in
>> behavior in Geronimo and not assume this change should be the concrete
>> fix.
>>
>> It would be bad to require our users to create these hidden classes
>> list when deploying their web app in geronimo, it would require too
>> much understanding of the Tuscany internals, and SCA is all about
>> simplicity.
>>
>> This thread in geronimo-dev [1] should give us more information on the
>> subject.
>>
>> [1] http://www.mail-archive.com/dev@geronimo.apache.org/msg65191.html
>>
>> On Mon, Sep 29, 2008 at 7:29 AM, <an...@apache.org> wrote:
>> > Author: antelder
>> > Date: Mon Sep 29 07:29:42 2008
>> > New Revision: 700138
>> >
>> > URL: http://svn.apache.org/viewvc?rev=700138&view=rev
>> > Log:
>> > TUSCANY-2622: Apply patch from Kevan Miller to Fix geronimo runtime
>> > errors caused by inverse-classloading in alert-aggregator-webapp demo
>> >
>> > Modified:
>> >
>> > tuscany/java/sca/demos/alert-aggregator-webapp/src/main/webapp/WEB-INF/geronimo-web.xml
>> >
>> > Modified:
>> > tuscany/java/sca/demos/alert-aggregator-webapp/src/main/webapp/WEB-INF/geronimo-web.xml
>> > URL:
>> > http://svn.apache.org/viewvc/tuscany/java/sca/demos/alert-aggregator-webapp/src/main/webapp/WEB-INF/geronimo-web.xml?rev=700138&r1=700137&r2=700138&view=diff
>> >
>> > ==============================================================================
>> > ---
>> > tuscany/java/sca/demos/alert-aggregator-webapp/src/main/webapp/WEB-INF/geronimo-web.xml
>> > (original)
>> > +++
>> > tuscany/java/sca/demos/alert-aggregator-webapp/src/main/webapp/WEB-INF/geronimo-web.xml
>> > Mon Sep 29 07:29:42 2008
>> > @@ -26,7 +26,12 @@
>> > <d:version>1.4-SNAPSHOT</d:version>
>> > <d:type>war</d:type>
>> > </d:moduleId>
>> > - <!-- TUSCANY-1986 -->
>> > - <d:inverse-classloading />
>> > + <!-- TUSCANY-2622 -->
>> > + <d:hidden-classes>
>> > + <d:filter>org.apache.axiom</d:filter>
>> > + <d:filter>org.apache.axis2</d:filter>
>> > + <d:filter>org.apache.commons</d:filter>
>> > + <d:filter>org.jdom</d:filter>
>> > + </d:hidden-classes>
>> > </d:environment>
>> > </web-app>
>> >
>> >
>> >
>>
>>
>>
>> --
>> Luciano Resende
>> Apache Tuscany, Apache PhotArk
>> http://people.apache.org/~lresende
>> http://lresende.blogspot.com/
>
>
--
Luciano Resende
Apache Tuscany, Apache PhotArk
http://people.apache.org/~lresende
http://lresende.blogspot.com/
Re: Inverse classloading versus Hidden class, was Re: svn commit: r700138 - /tuscany/java/sca/demos/alert-aggregator-webapp/src/main/webapp/WEB-INF/geronimo-web.xml
Posted by ant elder <an...@gmail.com>.
I don't disagree with you, and with the ongoing SCA JEE work I expect there
will be a more changes around this area. This fix gets things working for
now though which I found useful.
...ant
On Tue, Sep 30, 2008 at 3:38 AM, Luciano Resende <lu...@gmail.com>wrote:
> I think we should better understand what is causing this change in
> behavior in Geronimo and not assume this change should be the concrete
> fix.
>
> It would be bad to require our users to create these hidden classes
> list when deploying their web app in geronimo, it would require too
> much understanding of the Tuscany internals, and SCA is all about
> simplicity.
>
> This thread in geronimo-dev [1] should give us more information on the
> subject.
>
> [1] http://www.mail-archive.com/dev@geronimo.apache.org/msg65191.html
>
> On Mon, Sep 29, 2008 at 7:29 AM, <an...@apache.org> wrote:
> > Author: antelder
> > Date: Mon Sep 29 07:29:42 2008
> > New Revision: 700138
> >
> > URL: http://svn.apache.org/viewvc?rev=700138&view=rev
> > Log:
> > TUSCANY-2622: Apply patch from Kevan Miller to Fix geronimo runtime
> errors caused by inverse-classloading in alert-aggregator-webapp demo
> >
> > Modified:
> >
> tuscany/java/sca/demos/alert-aggregator-webapp/src/main/webapp/WEB-INF/geronimo-web.xml
> >
> > Modified:
> tuscany/java/sca/demos/alert-aggregator-webapp/src/main/webapp/WEB-INF/geronimo-web.xml
> > URL:
> http://svn.apache.org/viewvc/tuscany/java/sca/demos/alert-aggregator-webapp/src/main/webapp/WEB-INF/geronimo-web.xml?rev=700138&r1=700137&r2=700138&view=diff
> >
> ==============================================================================
> > ---
> tuscany/java/sca/demos/alert-aggregator-webapp/src/main/webapp/WEB-INF/geronimo-web.xml
> (original)
> > +++
> tuscany/java/sca/demos/alert-aggregator-webapp/src/main/webapp/WEB-INF/geronimo-web.xml
> Mon Sep 29 07:29:42 2008
> > @@ -26,7 +26,12 @@
> > <d:version>1.4-SNAPSHOT</d:version>
> > <d:type>war</d:type>
> > </d:moduleId>
> > - <!-- TUSCANY-1986 -->
> > - <d:inverse-classloading />
> > + <!-- TUSCANY-2622 -->
> > + <d:hidden-classes>
> > + <d:filter>org.apache.axiom</d:filter>
> > + <d:filter>org.apache.axis2</d:filter>
> > + <d:filter>org.apache.commons</d:filter>
> > + <d:filter>org.jdom</d:filter>
> > + </d:hidden-classes>
> > </d:environment>
> > </web-app>
> >
> >
> >
>
>
>
> --
> Luciano Resende
> Apache Tuscany, Apache PhotArk
> http://people.apache.org/~lresende <http://people.apache.org/%7Elresende>
> http://lresende.blogspot.com/
>