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 <[EMAIL PROTECTED]> 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 <[EMAIL PROTECTED]>Eran
> Chinthaka <[EMAIL PROTECTED]>
>
>
>                         *Eran Chinthaka <[EMAIL PROTECTED]>*
>
>                         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" <[EMAIL PROTECTED]>"Davanum
>  > Srinivas" <[EMAIL PROTECTED]>
>  >
>  >
>  >                         *"Davanum Srinivas" <[EMAIL PROTECTED]>*
>  >
>  >                         06/25/2007 09:29 AM
>  >                         Please respond to
>  >                         axis-dev@ws.apache.org
>  >
>  >
>  >
>  > To
>  >
>  > "Afkham Azeez" <[EMAIL PROTECTED]>
>  >
>  > 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 <[EMAIL PROTECTED]> 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 <[EMAIL PROTECTED]> 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:
>  >> [EMAIL PROTECTED]
>  >> > For additional commands, e-mail: [EMAIL PROTECTED]
>  >> >
>  >> >
>  >>
>  >>
>  >>
>  >> --
>  >> 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: [EMAIL PROTECTED]
>  > For additional commands, e-mail: [EMAIL PROTECTED]
>  >
>  >
>
> -----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: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>

--
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]




--
Davanum Srinivas :: http://davanum.wordpress.com

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to