You are viewing a plain text version of this content. The canonical link for it is here.
Posted to java-dev@axis.apache.org by Davanum Srinivas <da...@gmail.com> on 2007/06/24 13:48:59 UTC
[Axis2] Dispatchers in org.apache.axis2.engine package
Team,
Anyone object to moving the following classes to org.apache.axis2.dispatchers?
AddressingBasedDispatcher.java
HTTPLocationBasedDispatcher.java
InstanceDispatcher.java
RequestURIBasedDispatcher.java
RequestURIOperationDispatcher.java
SOAPActionBasedDispatcher.java
SOAPMessageBodyBasedDispatcher.java
Leaving AbstractDispatcher.java as-is in org.apache.axis2.engine
thanks,
dims
--
Davanum Srinivas :: http://davanum.wordpress.com
---------------------------------------------------------------------
To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-dev-help@ws.apache.org
Re: [Axis2] Dispatchers in org.apache.axis2.engine package
Posted by Davanum Srinivas <da...@gmail.com>.
Guess, i am trying to get the public API "right"...Right now there are
some *Dispatcher classes in org.apache.axis2.dispatchers and some
*Dispatchers in org.apache.axis2.engine.
-- dims
On 6/24/07, Sanjiva Weerawarana <sa...@opensource.lk> wrote:
> Also, aren't the package names of dispatchers sort of a "public API" bit
> of Axis2?? If so I'm against this unless there's something to be gained by it.
>
> Sanjiva.
>
> Eran Chinthaka wrote:
> > -----BEGIN PGP SIGNED MESSAGE-----
> > Hash: SHA1
> >
> > I have no objection to this but is there something you will gain by
> > doing this or rather is there a problem if we leave them as it is? This
> > is some what similar to the question Sanjiva was asking and had become
> > the theme for me when I am doing any change to Axis2. If something is
> > not broken, why bother trying to fix it and create probably un-necessary
> > trouble.
> >
> > The reason I am telling like that is, we are getting hammered for
> > ClassNotFoundExceptions and MethodNotFoundExceptions in the users list.
> > Simply we do too much of refactoring. Most users are now familiar with
> > some internals of Axis2 and not only those few public APIs.
> >
> > I am repeating my self, I have no objection to this, but just wanted to
> > give some ideas before any refactoring :)
> >
> > Chinthaka
> >
> > Davanum Srinivas wrote:
> >> Team,
> >>
> >> Anyone object to moving the following classes to
> >> org.apache.axis2.dispatchers?
> >>
> >> AddressingBasedDispatcher.java
> >> HTTPLocationBasedDispatcher.java
> >> InstanceDispatcher.java
> >> RequestURIBasedDispatcher.java
> >> RequestURIOperationDispatcher.java
> >> SOAPActionBasedDispatcher.java
> >> SOAPMessageBodyBasedDispatcher.java
> >>
> >> Leaving AbstractDispatcher.java as-is in org.apache.axis2.engine
> >>
> >> thanks,
> >> dims
> >>
> >
> > -----BEGIN PGP SIGNATURE-----
> > Version: GnuPG v1.4.3 (GNU/Linux)
> > Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
> >
> > iD8DBQFGfwFPjON2uBzUhh8RAp1YAJ0Q4daijAUeCOYuRpm8EcSxZNCj3gCgo/B5
> > 6R03kK0JHrZgS1zcQ86txxE=
> > =hzF6
> > -----END PGP SIGNATURE-----
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
> > For additional commands, e-mail: axis-dev-help@ws.apache.org
> >
> >
>
> --
> Sanjiva Weerawarana, Ph.D.
> Founder & Director; Lanka Software Foundation; http://www.opensource.lk/
> Founder, Chairman & CEO; WSO2, Inc.; http://www.wso2.com/
> Director; Open Source Initiative; http://www.opensource.org/
> Member; Apache Software Foundation; http://www.apache.org/
> Visiting Lecturer; University of Moratuwa; http://www.cse.mrt.ac.lk/
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
> For additional commands, e-mail: axis-dev-help@ws.apache.org
>
>
--
Davanum Srinivas :: http://davanum.wordpress.com
---------------------------------------------------------------------
To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-dev-help@ws.apache.org
Re: [Axis2] Dispatchers in org.apache.axis2.engine package
Posted by Sanjiva Weerawarana <sa...@opensource.lk>.
Also, aren't the package names of dispatchers sort of a "public API" bit
of Axis2?? If so I'm against this unless there's something to be gained by it.
Sanjiva.
Eran Chinthaka wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> I have no objection to this but is there something you will gain by
> doing this or rather is there a problem if we leave them as it is? This
> is some what similar to the question Sanjiva was asking and had become
> the theme for me when I am doing any change to Axis2. If something is
> not broken, why bother trying to fix it and create probably un-necessary
> trouble.
>
> The reason I am telling like that is, we are getting hammered for
> ClassNotFoundExceptions and MethodNotFoundExceptions in the users list.
> Simply we do too much of refactoring. Most users are now familiar with
> some internals of Axis2 and not only those few public APIs.
>
> I am repeating my self, I have no objection to this, but just wanted to
> give some ideas before any refactoring :)
>
> Chinthaka
>
> Davanum Srinivas wrote:
>> Team,
>>
>> Anyone object to moving the following classes to
>> org.apache.axis2.dispatchers?
>>
>> AddressingBasedDispatcher.java
>> HTTPLocationBasedDispatcher.java
>> InstanceDispatcher.java
>> RequestURIBasedDispatcher.java
>> RequestURIOperationDispatcher.java
>> SOAPActionBasedDispatcher.java
>> SOAPMessageBodyBasedDispatcher.java
>>
>> Leaving AbstractDispatcher.java as-is in org.apache.axis2.engine
>>
>> thanks,
>> dims
>>
>
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.3 (GNU/Linux)
> Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
>
> iD8DBQFGfwFPjON2uBzUhh8RAp1YAJ0Q4daijAUeCOYuRpm8EcSxZNCj3gCgo/B5
> 6R03kK0JHrZgS1zcQ86txxE=
> =hzF6
> -----END PGP SIGNATURE-----
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
> For additional commands, e-mail: axis-dev-help@ws.apache.org
>
>
--
Sanjiva Weerawarana, Ph.D.
Founder & Director; Lanka Software Foundation; http://www.opensource.lk/
Founder, Chairman & CEO; WSO2, Inc.; http://www.wso2.com/
Director; Open Source Initiative; http://www.opensource.org/
Member; Apache Software Foundation; http://www.apache.org/
Visiting Lecturer; University of Moratuwa; http://www.cse.mrt.ac.lk/
---------------------------------------------------------------------
To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-dev-help@ws.apache.org
Re: [Axis2] Dispatchers in org.apache.axis2.engine package
Posted by Eran Chinthaka <ch...@opensource.lk>.
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
I have no objection to this but is there something you will gain by
doing this or rather is there a problem if we leave them as it is? This
is some what similar to the question Sanjiva was asking and had become
the theme for me when I am doing any change to Axis2. If something is
not broken, why bother trying to fix it and create probably un-necessary
trouble.
The reason I am telling like that is, we are getting hammered for
ClassNotFoundExceptions and MethodNotFoundExceptions in the users list.
Simply we do too much of refactoring. Most users are now familiar with
some internals of Axis2 and not only those few public APIs.
I am repeating my self, I have no objection to this, but just wanted to
give some ideas before any refactoring :)
Chinthaka
Davanum Srinivas wrote:
> Team,
>
> Anyone object to moving the following classes to
> org.apache.axis2.dispatchers?
>
> AddressingBasedDispatcher.java
> HTTPLocationBasedDispatcher.java
> InstanceDispatcher.java
> RequestURIBasedDispatcher.java
> RequestURIOperationDispatcher.java
> SOAPActionBasedDispatcher.java
> SOAPMessageBodyBasedDispatcher.java
>
> Leaving AbstractDispatcher.java as-is in org.apache.axis2.engine
>
> thanks,
> dims
>
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.3 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iD8DBQFGfwFPjON2uBzUhh8RAp1YAJ0Q4daijAUeCOYuRpm8EcSxZNCj3gCgo/B5
6R03kK0JHrZgS1zcQ86txxE=
=hzF6
-----END PGP SIGNATURE-----
---------------------------------------------------------------------
To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-dev-help@ws.apache.org
Re: [Axis2] Dispatchers in org.apache.axis2.engine package
Posted by Glen Daniels <gl...@thoughtcraft.com>.
> So, IMO, lets move them, lets get the API/SPI right if we can (while
> maintaining backwards compatibility).
+1 to this, and to the "move, extend, and deprecate" pattern that's
being discussed.
--Glen
---------------------------------------------------------------------
To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-dev-help@ws.apache.org
Re: [Axis2] Dispatchers in org.apache.axis2.engine package
Posted by David Illsley <da...@gmail.com>.
Aaaargh.
Sorry, this is all my fault. I was trying to do the *right thing* and
have them in .engine but I keep on getting distracted. I think in
.dispatchers is the right place - I've always said I don't think any
user should have to look in the .engine package.
If we do move the dispatchers, my plan had been to leave shells in
.engine so that erxisting axis2.xmls would function (with the shalls
marked for removal in 1.4), document the move, and provide an xslt
transform to 'upgrade' the config files.To be truthful, it's doing the
last bit that's delayed me.
So, IMO, lets move them, lets get the API/SPI right if we can (while
maintaining backwards compatibility).
David
On 25/06/07, Afkham Azeez <af...@gmail.com> wrote:
> I am not in favor of this change. This is going to annoy a large number of
> users who may want to migrate from Axis2 1.2 to 1.3. They will have to edit
> all of their axis2.xml files. They may go thorugh a lot of pain to figure
> out that the package has to be changed in their axis2.xml files. The last
> thing we need at the moment is unhappy users. This is a mistake we may have
> to live with.
>
> -- Azeez
>
>
> On 6/24/07, Davanum Srinivas <da...@gmail.com> wrote:
> > Team,
> >
> > Anyone object to moving the following classes to
> org.apache.axis2.dispatchers?
> >
> > AddressingBasedDispatcher.java
> > HTTPLocationBasedDispatcher.java
> > InstanceDispatcher.java
> > RequestURIBasedDispatcher.java
> > RequestURIOperationDispatcher.java
> > SOAPActionBasedDispatcher.java
> > SOAPMessageBodyBasedDispatcher.java
> >
> > Leaving AbstractDispatcher.java as-is in org.apache.axis2.engine
> >
> > thanks,
> > dims
> >
> > --
> > Davanum Srinivas :: http://davanum.wordpress.com
> >
> >
> ---------------------------------------------------------------------
> > To unsubscribe, e-mail:
> axis-dev-unsubscribe@ws.apache.org
> > For additional commands, e-mail: axis-dev-help@ws.apache.org
> >
> >
>
>
>
> --
> Thanks
> Afkham Azeez
>
> http://www.wso2.org
> GPG Fingerprint: 643F C2AF EB78 F886 40C9 B2A2 4AE2 C887 665E 0760
--
David Illsley - IBM Web Services Development
---------------------------------------------------------------------
To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-dev-help@ws.apache.org
Re: [Axis2] Dispatchers in org.apache.axis2.engine package
Posted by Davanum Srinivas <da...@gmail.com>.
Hehe, that's what i am doing right now :)
-- dims
On 6/25/07, Eran Chinthaka <ch...@opensource.lk> wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> I can remember a trick that we have used for UUID generator. How about
> the following.
>
> package o.a.a.engine;
>
> /**
> * This class is moved to o.a.a.dispatchers.FooDispatcher
> * @deprecated
> */
> public class FooDispatcher extends o.a.a.dispatchers.FooDispatcher{
> }
>
> Chinthaka
>
> Nicholas L Gallardo wrote:
> > +1 That sounds like the best solution here. I agree with David's comment
> > that user's shouldn't be accessing anything inside of the .engine
> > package either. If it's part of an official API, move it to other package.
> >
> > Dims, what about adding something that issues a deprecation warning when
> > the dispatcher loaded contains the "org.apache.axis2.engine" package.
> > Depending on how long the internal is between 1.4 and 1.3, you could
> > then remove it in the next release or wait one more depending on how
> > long you wanted to support it.
> >
> > Regards,
> >
> > -Nick
> >
> > Inactive hide details for "Davanum Srinivas" <da...@gmail.com>"Davanum
> > Srinivas" <da...@gmail.com>
> >
> >
> > *"Davanum Srinivas" <da...@gmail.com>*
> >
> > 06/25/2007 09:29 AM
> > Please respond to
> > axis-dev@ws.apache.org
> >
> >
> >
> > To
> >
> > "Afkham Azeez" <af...@gmail.com>
> >
> > cc
> >
> > axis-dev@ws.apache.org
> >
> > Subject
> >
> > Re: [Axis2] Dispatchers in org.apache.axis2.engine package
> >
> >
> >
> >
> > So, if i come up with a technique for backward compat w/o touching the
> > old axis2.xml files, that should be ok right?
> >
> > thanks,
> > dims
> >
> > On 6/25/07, Afkham Azeez <af...@gmail.com> wrote:
> >> I am not in favor of this change. This is going to annoy a large number of
> >> users who may want to migrate from Axis2 1.2 to 1.3. They will have to
> > edit
> >> all of their axis2.xml files. They may go thorugh a lot of pain to figure
> >> out that the package has to be changed in their axis2.xml files. The last
> >> thing we need at the moment is unhappy users. This is a mistake we may
> > have
> >> to live with.
> >>
> >> -- Azeez
> >>
> >>
> >> On 6/24/07, Davanum Srinivas <da...@gmail.com> wrote:
> >> >
> >> > Team,
> >> >
> >> > Anyone object to moving the following classes to
> >> org.apache.axis2.dispatchers?
> >> >
> >> > AddressingBasedDispatcher.java
> >> > HTTPLocationBasedDispatcher.java
> >> > InstanceDispatcher.java
> >> > RequestURIBasedDispatcher.java
> >> > RequestURIOperationDispatcher.java
> >> > SOAPActionBasedDispatcher.java
> >> > SOAPMessageBodyBasedDispatcher.java
> >> >
> >> > Leaving AbstractDispatcher.java as-is in org.apache.axis2.engine
> >> >
> >> > thanks,
> >> > dims
> >> >
> >> > --
> >> > Davanum Srinivas :: http://davanum.wordpress.com
> >> >
> >> >
> >> ---------------------------------------------------------------------
> >> > To unsubscribe, e-mail:
> >> axis-dev-unsubscribe@ws.apache.org
> >> > For additional commands, e-mail: axis-dev-help@ws.apache.org
> >> >
> >> >
> >>
> >>
> >>
> >> --
> >> Thanks
> >> Afkham Azeez
> >>
> >> http://www.wso2.org
> >> GPG Fingerprint: 643F C2AF EB78 F886 40C9 B2A2 4AE2 C887 665E 0760
> >
> >
> > --
> > Davanum Srinivas :: http://davanum.wordpress.com
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
> > For additional commands, e-mail: axis-dev-help@ws.apache.org
> >
> >
>
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.3 (GNU/Linux)
> Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
>
> iD4DBQFGf+LsjON2uBzUhh8RArmJAJij7mXQDSKYDyCY1RVxEAFs2ouGAKCphJJ0
> R8g0IaSfmN/QduNHp4i8XA==
> =qDVE
> -----END PGP SIGNATURE-----
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
> For additional commands, e-mail: axis-dev-help@ws.apache.org
>
>
--
Davanum Srinivas :: http://davanum.wordpress.com
---------------------------------------------------------------------
To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-dev-help@ws.apache.org
Re: [Axis2] Dispatchers in org.apache.axis2.engine package
Posted by Davanum Srinivas <da...@gmail.com>.
Folks,
Please review the changes i made today. FYI, here's a quick html where
you can view all the commits (a week's worth).
http://ws.zones.apache.org/~dims/axis2/
thanks,
dims
On 6/25/07, Sanjiva Weerawarana <sa...@opensource.lk> wrote:
> +1 for this change as long as we have the backwards compat capability.
>
> Sanjiva.
>
> Nicholas L Gallardo wrote:
> > Chintaka,
> >
> > +1
> >
> > That's exactly what I was thinking as well...
> >
> > -Nick
> >
> >
> > Inactive hide details for Eran Chinthaka <ch...@opensource.lk>Eran
> > Chinthaka <ch...@opensource.lk>
> >
> >
> > *Eran Chinthaka <ch...@opensource.lk>*
> >
> > 06/25/2007 10:44 AM
> > Please respond to
> > axis-dev@ws.apache.org
> >
> >
> >
> > To
> >
> > axis-dev@ws.apache.org
> >
> > cc
> >
> >
> > Subject
> >
> > Re: [Axis2] Dispatchers in org.apache.axis2.engine package
> >
> >
> >
> >
> > -----BEGIN PGP SIGNED MESSAGE-----
> > Hash: SHA1
> >
> > I can remember a trick that we have used for UUID generator. How about
> > the following.
> >
> > package o.a.a.engine;
> >
> > /**
> > * This class is moved to o.a.a.dispatchers.FooDispatcher
> > * @deprecated
> > */
> > public class FooDispatcher extends o.a.a.dispatchers.FooDispatcher{
> > }
> >
> > Chinthaka
> >
> > Nicholas L Gallardo wrote:
> > > +1 That sounds like the best solution here. I agree with David's comment
> > > that user's shouldn't be accessing anything inside of the .engine
> > > package either. If it's part of an official API, move it to other
> > package.
> > >
> > > Dims, what about adding something that issues a deprecation warning when
> > > the dispatcher loaded contains the "org.apache.axis2.engine" package.
> > > Depending on how long the internal is between 1.4 and 1.3, you could
> > > then remove it in the next release or wait one more depending on how
> > > long you wanted to support it.
> > >
> > > Regards,
> > >
> > > -Nick
> > >
> > > Inactive hide details for "Davanum Srinivas" <da...@gmail.com>"Davanum
> > > Srinivas" <da...@gmail.com>
> > >
> > >
> > > *"Davanum Srinivas" <da...@gmail.com>*
> > >
> > > 06/25/2007 09:29 AM
> > > Please respond to
> > > axis-dev@ws.apache.org
> > >
> > >
> > >
> > > To
> > >
> > > "Afkham Azeez" <af...@gmail.com>
> > >
> > > cc
> > >
> > > axis-dev@ws.apache.org
> > >
> > > Subject
> > >
> > > Re: [Axis2] Dispatchers in org.apache.axis2.engine package
> > >
> > >
> > >
> > >
> > > So, if i come up with a technique for backward compat w/o touching the
> > > old axis2.xml files, that should be ok right?
> > >
> > > thanks,
> > > dims
> > >
> > > On 6/25/07, Afkham Azeez <af...@gmail.com> wrote:
> > >> I am not in favor of this change. This is going to annoy a large
> > number of
> > >> users who may want to migrate from Axis2 1.2 to 1.3. They will have to
> > > edit
> > >> all of their axis2.xml files. They may go thorugh a lot of pain to
> > figure
> > >> out that the package has to be changed in their axis2.xml files. The
> > last
> > >> thing we need at the moment is unhappy users. This is a mistake we may
> > > have
> > >> to live with.
> > >>
> > >> -- Azeez
> > >>
> > >>
> > >> On 6/24/07, Davanum Srinivas <da...@gmail.com> wrote:
> > >> >
> > >> > Team,
> > >> >
> > >> > Anyone object to moving the following classes to
> > >> org.apache.axis2.dispatchers?
> > >> >
> > >> > AddressingBasedDispatcher.java
> > >> > HTTPLocationBasedDispatcher.java
> > >> > InstanceDispatcher.java
> > >> > RequestURIBasedDispatcher.java
> > >> > RequestURIOperationDispatcher.java
> > >> > SOAPActionBasedDispatcher.java
> > >> > SOAPMessageBodyBasedDispatcher.java
> > >> >
> > >> > Leaving AbstractDispatcher.java as-is in org.apache.axis2.engine
> > >> >
> > >> > thanks,
> > >> > dims
> > >> >
> > >> > --
> > >> > Davanum Srinivas :: http://davanum.wordpress.com
> > >> >
> > >> >
> > >> ---------------------------------------------------------------------
> > >> > To unsubscribe, e-mail:
> > >> axis-dev-unsubscribe@ws.apache.org
> > >> > For additional commands, e-mail: axis-dev-help@ws.apache.org
> > >> >
> > >> >
> > >>
> > >>
> > >>
> > >> --
> > >> Thanks
> > >> Afkham Azeez
> > >>
> > >> http://www.wso2.org
> > >> GPG Fingerprint: 643F C2AF EB78 F886 40C9 B2A2 4AE2 C887 665E 0760
> > >
> > >
> > > --
> > > Davanum Srinivas :: http://davanum.wordpress.com
> > >
> > > ---------------------------------------------------------------------
> > > To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
> > > For additional commands, e-mail: axis-dev-help@ws.apache.org
> > >
> > >
> >
> > -----BEGIN PGP SIGNATURE-----
> > Version: GnuPG v1.4.3 (GNU/Linux)
> > Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
> >
> > iD4DBQFGf+LsjON2uBzUhh8RArmJAJij7mXQDSKYDyCY1RVxEAFs2ouGAKCphJJ0
> > R8g0IaSfmN/QduNHp4i8XA==
> > =qDVE
> > -----END PGP SIGNATURE-----
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
> > For additional commands, e-mail: axis-dev-help@ws.apache.org
> >
> >
>
> --
> Sanjiva Weerawarana, Ph.D.
> Founder & Director; Lanka Software Foundation; http://www.opensource.lk/
> Founder, Chairman & CEO; WSO2, Inc.; http://www.wso2.com/
> Director; Open Source Initiative; http://www.opensource.org/
> Member; Apache Software Foundation; http://www.apache.org/
> Visiting Lecturer; University of Moratuwa; http://www.cse.mrt.ac.lk/
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
> For additional commands, e-mail: axis-dev-help@ws.apache.org
>
>
--
Davanum Srinivas :: http://davanum.wordpress.com
---------------------------------------------------------------------
To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-dev-help@ws.apache.org
Re: [Axis2] Dispatchers in org.apache.axis2.engine package
Posted by Sanjiva Weerawarana <sa...@opensource.lk>.
+1 for this change as long as we have the backwards compat capability.
Sanjiva.
Nicholas L Gallardo wrote:
> Chintaka,
>
> +1
>
> That's exactly what I was thinking as well...
>
> -Nick
>
>
> Inactive hide details for Eran Chinthaka <ch...@opensource.lk>Eran
> Chinthaka <ch...@opensource.lk>
>
>
> *Eran Chinthaka <ch...@opensource.lk>*
>
> 06/25/2007 10:44 AM
> Please respond to
> axis-dev@ws.apache.org
>
>
>
> To
>
> axis-dev@ws.apache.org
>
> cc
>
>
> Subject
>
> Re: [Axis2] Dispatchers in org.apache.axis2.engine package
>
>
>
>
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> I can remember a trick that we have used for UUID generator. How about
> the following.
>
> package o.a.a.engine;
>
> /**
> * This class is moved to o.a.a.dispatchers.FooDispatcher
> * @deprecated
> */
> public class FooDispatcher extends o.a.a.dispatchers.FooDispatcher{
> }
>
> Chinthaka
>
> Nicholas L Gallardo wrote:
> > +1 That sounds like the best solution here. I agree with David's comment
> > that user's shouldn't be accessing anything inside of the .engine
> > package either. If it's part of an official API, move it to other
> package.
> >
> > Dims, what about adding something that issues a deprecation warning when
> > the dispatcher loaded contains the "org.apache.axis2.engine" package.
> > Depending on how long the internal is between 1.4 and 1.3, you could
> > then remove it in the next release or wait one more depending on how
> > long you wanted to support it.
> >
> > Regards,
> >
> > -Nick
> >
> > Inactive hide details for "Davanum Srinivas" <da...@gmail.com>"Davanum
> > Srinivas" <da...@gmail.com>
> >
> >
> > *"Davanum Srinivas" <da...@gmail.com>*
> >
> > 06/25/2007 09:29 AM
> > Please respond to
> > axis-dev@ws.apache.org
> >
> >
> >
> > To
> >
> > "Afkham Azeez" <af...@gmail.com>
> >
> > cc
> >
> > axis-dev@ws.apache.org
> >
> > Subject
> >
> > Re: [Axis2] Dispatchers in org.apache.axis2.engine package
> >
> >
> >
> >
> > So, if i come up with a technique for backward compat w/o touching the
> > old axis2.xml files, that should be ok right?
> >
> > thanks,
> > dims
> >
> > On 6/25/07, Afkham Azeez <af...@gmail.com> wrote:
> >> I am not in favor of this change. This is going to annoy a large
> number of
> >> users who may want to migrate from Axis2 1.2 to 1.3. They will have to
> > edit
> >> all of their axis2.xml files. They may go thorugh a lot of pain to
> figure
> >> out that the package has to be changed in their axis2.xml files. The
> last
> >> thing we need at the moment is unhappy users. This is a mistake we may
> > have
> >> to live with.
> >>
> >> -- Azeez
> >>
> >>
> >> On 6/24/07, Davanum Srinivas <da...@gmail.com> wrote:
> >> >
> >> > Team,
> >> >
> >> > Anyone object to moving the following classes to
> >> org.apache.axis2.dispatchers?
> >> >
> >> > AddressingBasedDispatcher.java
> >> > HTTPLocationBasedDispatcher.java
> >> > InstanceDispatcher.java
> >> > RequestURIBasedDispatcher.java
> >> > RequestURIOperationDispatcher.java
> >> > SOAPActionBasedDispatcher.java
> >> > SOAPMessageBodyBasedDispatcher.java
> >> >
> >> > Leaving AbstractDispatcher.java as-is in org.apache.axis2.engine
> >> >
> >> > thanks,
> >> > dims
> >> >
> >> > --
> >> > Davanum Srinivas :: http://davanum.wordpress.com
> >> >
> >> >
> >> ---------------------------------------------------------------------
> >> > To unsubscribe, e-mail:
> >> axis-dev-unsubscribe@ws.apache.org
> >> > For additional commands, e-mail: axis-dev-help@ws.apache.org
> >> >
> >> >
> >>
> >>
> >>
> >> --
> >> Thanks
> >> Afkham Azeez
> >>
> >> http://www.wso2.org
> >> GPG Fingerprint: 643F C2AF EB78 F886 40C9 B2A2 4AE2 C887 665E 0760
> >
> >
> > --
> > Davanum Srinivas :: http://davanum.wordpress.com
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
> > For additional commands, e-mail: axis-dev-help@ws.apache.org
> >
> >
>
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.3 (GNU/Linux)
> Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
>
> iD4DBQFGf+LsjON2uBzUhh8RArmJAJij7mXQDSKYDyCY1RVxEAFs2ouGAKCphJJ0
> R8g0IaSfmN/QduNHp4i8XA==
> =qDVE
> -----END PGP SIGNATURE-----
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
> For additional commands, e-mail: axis-dev-help@ws.apache.org
>
>
--
Sanjiva Weerawarana, Ph.D.
Founder & Director; Lanka Software Foundation; http://www.opensource.lk/
Founder, Chairman & CEO; WSO2, Inc.; http://www.wso2.com/
Director; Open Source Initiative; http://www.opensource.org/
Member; Apache Software Foundation; http://www.apache.org/
Visiting Lecturer; University of Moratuwa; http://www.cse.mrt.ac.lk/
---------------------------------------------------------------------
To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-dev-help@ws.apache.org
Re: [Axis2] Dispatchers in org.apache.axis2.engine package
Posted by Nicholas L Gallardo <nl...@us.ibm.com>.
Chintaka,
+1
That's exactly what I was thinking as well...
-Nick
Eran Chinthaka
<chinthaka@openso
urce.lk> To
axis-dev@ws.apache.org
06/25/2007 10:44 cc
AM
Subject
Re: [Axis2] Dispatchers in
Please respond to org.apache.axis2.engine package
axis-dev@ws.apach
e.org
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
I can remember a trick that we have used for UUID generator. How about
the following.
package o.a.a.engine;
/**
* This class is moved to o.a.a.dispatchers.FooDispatcher
* @deprecated
*/
public class FooDispatcher extends o.a.a.dispatchers.FooDispatcher{
}
Chinthaka
Nicholas L Gallardo wrote:
> +1 That sounds like the best solution here. I agree with David's comment
> that user's shouldn't be accessing anything inside of the .engine
> package either. If it's part of an official API, move it to other
package.
>
> Dims, what about adding something that issues a deprecation warning when
> the dispatcher loaded contains the "org.apache.axis2.engine" package.
> Depending on how long the internal is between 1.4 and 1.3, you could
> then remove it in the next release or wait one more depending on how
> long you wanted to support it.
>
> Regards,
>
> -Nick
>
> Inactive hide details for "Davanum Srinivas" <da...@gmail.com>"Davanum
> Srinivas" <da...@gmail.com>
>
>
> *"Davanum Srinivas" <da...@gmail.com>*
>
> 06/25/2007 09:29 AM
> Please respond to
> axis-dev@ws.apache.org
>
>
>
> To
>
> "Afkham Azeez" <af...@gmail.com>
>
> cc
>
> axis-dev@ws.apache.org
>
> Subject
>
> Re: [Axis2] Dispatchers in org.apache.axis2.engine package
>
>
>
>
> So, if i come up with a technique for backward compat w/o touching the
> old axis2.xml files, that should be ok right?
>
> thanks,
> dims
>
> On 6/25/07, Afkham Azeez <af...@gmail.com> wrote:
>> I am not in favor of this change. This is going to annoy a large number
of
>> users who may want to migrate from Axis2 1.2 to 1.3. They will have to
> edit
>> all of their axis2.xml files. They may go thorugh a lot of pain to
figure
>> out that the package has to be changed in their axis2.xml files. The
last
>> thing we need at the moment is unhappy users. This is a mistake we may
> have
>> to live with.
>>
>> -- Azeez
>>
>>
>> On 6/24/07, Davanum Srinivas <da...@gmail.com> wrote:
>> >
>> > Team,
>> >
>> > Anyone object to moving the following classes to
>> org.apache.axis2.dispatchers?
>> >
>> > AddressingBasedDispatcher.java
>> > HTTPLocationBasedDispatcher.java
>> > InstanceDispatcher.java
>> > RequestURIBasedDispatcher.java
>> > RequestURIOperationDispatcher.java
>> > SOAPActionBasedDispatcher.java
>> > SOAPMessageBodyBasedDispatcher.java
>> >
>> > Leaving AbstractDispatcher.java as-is in org.apache.axis2.engine
>> >
>> > thanks,
>> > dims
>> >
>> > --
>> > Davanum Srinivas :: http://davanum.wordpress.com
>> >
>> >
>> ---------------------------------------------------------------------
>> > To unsubscribe, e-mail:
>> axis-dev-unsubscribe@ws.apache.org
>> > For additional commands, e-mail: axis-dev-help@ws.apache.org
>> >
>> >
>>
>>
>>
>> --
>> Thanks
>> Afkham Azeez
>>
>> http://www.wso2.org
>> GPG Fingerprint: 643F C2AF EB78 F886 40C9 B2A2 4AE2 C887 665E 0760
>
>
> --
> Davanum Srinivas :: http://davanum.wordpress.com
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
> For additional commands, e-mail: axis-dev-help@ws.apache.org
>
>
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.3 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iD4DBQFGf+LsjON2uBzUhh8RArmJAJij7mXQDSKYDyCY1RVxEAFs2ouGAKCphJJ0
R8g0IaSfmN/QduNHp4i8XA==
=qDVE
-----END PGP SIGNATURE-----
---------------------------------------------------------------------
To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-dev-help@ws.apache.org
Re: [Axis2] Dispatchers in org.apache.axis2.engine package
Posted by Eran Chinthaka <ch...@opensource.lk>.
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
I can remember a trick that we have used for UUID generator. How about
the following.
package o.a.a.engine;
/**
* This class is moved to o.a.a.dispatchers.FooDispatcher
* @deprecated
*/
public class FooDispatcher extends o.a.a.dispatchers.FooDispatcher{
}
Chinthaka
Nicholas L Gallardo wrote:
> +1 That sounds like the best solution here. I agree with David's comment
> that user's shouldn't be accessing anything inside of the .engine
> package either. If it's part of an official API, move it to other package.
>
> Dims, what about adding something that issues a deprecation warning when
> the dispatcher loaded contains the "org.apache.axis2.engine" package.
> Depending on how long the internal is between 1.4 and 1.3, you could
> then remove it in the next release or wait one more depending on how
> long you wanted to support it.
>
> Regards,
>
> -Nick
>
> Inactive hide details for "Davanum Srinivas" <da...@gmail.com>"Davanum
> Srinivas" <da...@gmail.com>
>
>
> *"Davanum Srinivas" <da...@gmail.com>*
>
> 06/25/2007 09:29 AM
> Please respond to
> axis-dev@ws.apache.org
>
>
>
> To
>
> "Afkham Azeez" <af...@gmail.com>
>
> cc
>
> axis-dev@ws.apache.org
>
> Subject
>
> Re: [Axis2] Dispatchers in org.apache.axis2.engine package
>
>
>
>
> So, if i come up with a technique for backward compat w/o touching the
> old axis2.xml files, that should be ok right?
>
> thanks,
> dims
>
> On 6/25/07, Afkham Azeez <af...@gmail.com> wrote:
>> I am not in favor of this change. This is going to annoy a large number of
>> users who may want to migrate from Axis2 1.2 to 1.3. They will have to
> edit
>> all of their axis2.xml files. They may go thorugh a lot of pain to figure
>> out that the package has to be changed in their axis2.xml files. The last
>> thing we need at the moment is unhappy users. This is a mistake we may
> have
>> to live with.
>>
>> -- Azeez
>>
>>
>> On 6/24/07, Davanum Srinivas <da...@gmail.com> wrote:
>> >
>> > Team,
>> >
>> > Anyone object to moving the following classes to
>> org.apache.axis2.dispatchers?
>> >
>> > AddressingBasedDispatcher.java
>> > HTTPLocationBasedDispatcher.java
>> > InstanceDispatcher.java
>> > RequestURIBasedDispatcher.java
>> > RequestURIOperationDispatcher.java
>> > SOAPActionBasedDispatcher.java
>> > SOAPMessageBodyBasedDispatcher.java
>> >
>> > Leaving AbstractDispatcher.java as-is in org.apache.axis2.engine
>> >
>> > thanks,
>> > dims
>> >
>> > --
>> > Davanum Srinivas :: http://davanum.wordpress.com
>> >
>> >
>> ---------------------------------------------------------------------
>> > To unsubscribe, e-mail:
>> axis-dev-unsubscribe@ws.apache.org
>> > For additional commands, e-mail: axis-dev-help@ws.apache.org
>> >
>> >
>>
>>
>>
>> --
>> Thanks
>> Afkham Azeez
>>
>> http://www.wso2.org
>> GPG Fingerprint: 643F C2AF EB78 F886 40C9 B2A2 4AE2 C887 665E 0760
>
>
> --
> Davanum Srinivas :: http://davanum.wordpress.com
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
> For additional commands, e-mail: axis-dev-help@ws.apache.org
>
>
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.3 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iD4DBQFGf+LsjON2uBzUhh8RArmJAJij7mXQDSKYDyCY1RVxEAFs2ouGAKCphJJ0
R8g0IaSfmN/QduNHp4i8XA==
=qDVE
-----END PGP SIGNATURE-----
---------------------------------------------------------------------
To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-dev-help@ws.apache.org
Re: [Axis2] Dispatchers in org.apache.axis2.engine package
Posted by Nicholas L Gallardo <nl...@us.ibm.com>.
+1 That sounds like the best solution here. I agree with David's comment
that user's shouldn't be accessing anything inside of the .engine package
either. If it's part of an official API, move it to other package.
Dims, what about adding something that issues a deprecation warning when
the dispatcher loaded contains the "org.apache.axis2.engine" package.
Depending on how long the internal is between 1.4 and 1.3, you could then
remove it in the next release or wait one more depending on how long you
wanted to support it.
Regards,
-Nick
"Davanum
Srinivas"
<davanum@gmail.co To
m> "Afkham Azeez" <af...@gmail.com>
cc
06/25/2007 09:29 axis-dev@ws.apache.org
AM Subject
Re: [Axis2] Dispatchers in
org.apache.axis2.engine package
Please respond to
axis-dev@ws.apach
e.org
So, if i come up with a technique for backward compat w/o touching the
old axis2.xml files, that should be ok right?
thanks,
dims
On 6/25/07, Afkham Azeez <af...@gmail.com> wrote:
> I am not in favor of this change. This is going to annoy a large number
of
> users who may want to migrate from Axis2 1.2 to 1.3. They will have to
edit
> all of their axis2.xml files. They may go thorugh a lot of pain to figure
> out that the package has to be changed in their axis2.xml files. The last
> thing we need at the moment is unhappy users. This is a mistake we may
have
> to live with.
>
> -- Azeez
>
>
> On 6/24/07, Davanum Srinivas <da...@gmail.com> wrote:
> >
> > Team,
> >
> > Anyone object to moving the following classes to
> org.apache.axis2.dispatchers?
> >
> > AddressingBasedDispatcher.java
> > HTTPLocationBasedDispatcher.java
> > InstanceDispatcher.java
> > RequestURIBasedDispatcher.java
> > RequestURIOperationDispatcher.java
> > SOAPActionBasedDispatcher.java
> > SOAPMessageBodyBasedDispatcher.java
> >
> > Leaving AbstractDispatcher.java as-is in org.apache.axis2.engine
> >
> > thanks,
> > dims
> >
> > --
> > Davanum Srinivas :: http://davanum.wordpress.com
> >
> >
> ---------------------------------------------------------------------
> > To unsubscribe, e-mail:
> axis-dev-unsubscribe@ws.apache.org
> > For additional commands, e-mail: axis-dev-help@ws.apache.org
> >
> >
>
>
>
> --
> Thanks
> Afkham Azeez
>
> http://www.wso2.org
> GPG Fingerprint: 643F C2AF EB78 F886 40C9 B2A2 4AE2 C887 665E 0760
--
Davanum Srinivas :: http://davanum.wordpress.com
---------------------------------------------------------------------
To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-dev-help@ws.apache.org
Re: [Axis2] Dispatchers in org.apache.axis2.engine package
Posted by Davanum Srinivas <da...@gmail.com>.
So, if i come up with a technique for backward compat w/o touching the
old axis2.xml files, that should be ok right?
thanks,
dims
On 6/25/07, Afkham Azeez <af...@gmail.com> wrote:
> I am not in favor of this change. This is going to annoy a large number of
> users who may want to migrate from Axis2 1.2 to 1.3. They will have to edit
> all of their axis2.xml files. They may go thorugh a lot of pain to figure
> out that the package has to be changed in their axis2.xml files. The last
> thing we need at the moment is unhappy users. This is a mistake we may have
> to live with.
>
> -- Azeez
>
>
> On 6/24/07, Davanum Srinivas <da...@gmail.com> wrote:
> >
> > Team,
> >
> > Anyone object to moving the following classes to
> org.apache.axis2.dispatchers?
> >
> > AddressingBasedDispatcher.java
> > HTTPLocationBasedDispatcher.java
> > InstanceDispatcher.java
> > RequestURIBasedDispatcher.java
> > RequestURIOperationDispatcher.java
> > SOAPActionBasedDispatcher.java
> > SOAPMessageBodyBasedDispatcher.java
> >
> > Leaving AbstractDispatcher.java as-is in org.apache.axis2.engine
> >
> > thanks,
> > dims
> >
> > --
> > Davanum Srinivas :: http://davanum.wordpress.com
> >
> >
> ---------------------------------------------------------------------
> > To unsubscribe, e-mail:
> axis-dev-unsubscribe@ws.apache.org
> > For additional commands, e-mail: axis-dev-help@ws.apache.org
> >
> >
>
>
>
> --
> Thanks
> Afkham Azeez
>
> http://www.wso2.org
> GPG Fingerprint: 643F C2AF EB78 F886 40C9 B2A2 4AE2 C887 665E 0760
--
Davanum Srinivas :: http://davanum.wordpress.com
---------------------------------------------------------------------
To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-dev-help@ws.apache.org
Re: [Axis2] Dispatchers in org.apache.axis2.engine package
Posted by Afkham Azeez <af...@gmail.com>.
I am not in favor of this change. This is going to annoy a large number of
users who may want to migrate from Axis2 1.2 to 1.3. They will have to edit
all of their axis2.xml files. They may go thorugh a lot of pain to figure
out that the package has to be changed in their axis2.xml files. The last
thing we need at the moment is unhappy users. This is a mistake we may have
to live with.
-- Azeez
On 6/24/07, Davanum Srinivas <da...@gmail.com> wrote:
>
> Team,
>
> Anyone object to moving the following classes to
> org.apache.axis2.dispatchers?
>
> AddressingBasedDispatcher.java
> HTTPLocationBasedDispatcher.java
> InstanceDispatcher.java
> RequestURIBasedDispatcher.java
> RequestURIOperationDispatcher.java
> SOAPActionBasedDispatcher.java
> SOAPMessageBodyBasedDispatcher.java
>
> Leaving AbstractDispatcher.java as-is in org.apache.axis2.engine
>
> thanks,
> dims
>
> --
> Davanum Srinivas :: http://davanum.wordpress.com
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
> For additional commands, e-mail: axis-dev-help@ws.apache.org
>
>
--
Thanks
Afkham Azeez
http://www.wso2.org
GPG Fingerprint: 643F C2AF EB78 F886 40C9 B2A2 4AE2 C887 665E 0760