Ok, great. I had a feeling we must have had a disconnect somewhere :-)
I'm more than happy to add a disclaimer in the comments.
Thanks,
Lori.
Lori Van Gulick
WebSphere Development
IBM Austin
(512) 838-7929
| Sanjiva Weerawarana <[EMAIL PROTECTED]>
08/18/2006 12:18 AM
|
|
Ouch. Sorry Lori & Dims: I missed what this change was impacting. As
this does not touch ServiceClient and is just adding two new classes to
do this I'm absolutely fine with it. Please consider my -1 lifted.
Lori can you please put some comments there saying that this is a
temporary solution which we may improve.
Thanks,
Sanjiva.
On Thu, 2006-08-17 at 08:42 -0400, Davanum Srinivas wrote:
> Sanjiva,
>
> If the concern is about "API Change" Please *review* the API change:
> http://ws.zones.apache.org/~dims/r431798/newdocs/changes.html
>
> Please note that there are outstanding issue for multiple services in
> wsdl. We have to start somewhere...
> http://issues.apache.org/jira/browse/AXIS2-352
>
> We can put a big disclaimer on the 2 new classes which will serve the
> purpose if dissuading folks who may try to use it. Other than that, it
> looks ok to me.
>
> thanks,
> dims
>
> On 8/17/06, Sanjiva Weerawarana <[EMAIL PROTECTED]> wrote:
> > On Wed, 2006-08-16 at 14:43 -0500, Lori Van Gulick wrote:
> > >
> > > Vote: Accept the changes for JIRA-965 as an interim solution.
> >
> > I don't like this .. this is an API change and once we do it we can't
> > remove it.
> >
> > So I'd prefer to fix it properly and am -1 for it.
> >
> > Do you guys need this for 1.1 or can it wait until after 1.1? Why can
> > there not be a private API that does this same function but does not go
> > into the main API right now?
> >
> > Ajith's point is relevant too- unless this is followed thru to codegen
> > then I don't see why a separate class can't be used.
> >
> > (I'm sorry I haven't been more responsive .. I'm been totally swamped
> > with ApacheCon Asia stuff and just haven't been able to keep up.)
> >
> > Sanjiva.
> >
> >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: [EMAIL PROTECTED]
> > For additional commands, e-mail: [EMAIL PROTECTED]
> >
> >
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
