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]
