axis-java-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Davanum Srinivas <dava...@gmail.com>
Subject Re: Good to GO!!!
Date Thu, 08 Jul 2004 23:50:45 GMT
Glen,

If you can show me how RPCParam is exposed to Axis users, i'll either
fix the situation by some refactoring magic (or) by rolling everything
back. I am not planning to make any more checkins till this issue is
resolved one way or another and the release finalized.

thanks,
dims

On Thu, 8 Jul 2004 17:35:07 -0400, Glen Daniels
<gdaniels@sonicsoftware.com> wrote:
> 
> Hm... I was going to say that the output param situation is common
> enough that that might be a showstopper on its own, but looking at
> Call.java:2462, it appears that we don't actually even put the RPCParam
> on the outputParams list?!?  I'm confused.
> 
> --Glen
> 
> 
> 
> > -----Original Message-----
> > From: Davanum Srinivas [mailto:davanum@gmail.com]
> > Sent: Thursday, July 08, 2004 5:29 PM
> > To: axis-dev@ws.apache.org
> > Cc: tomj@macromedia.com; axis-tck@ws.apache.org; dims@apache.org
> > Subject: Re: Good to GO!!!
> >
> > Glen,
> >
> > Do you have a problem with just the "call.getOutputParams()"
> > situation? i don't see any others that expose RPCParam....
> >
> > thanks,
> > dims
> >
> > On Thu, 8 Jul 2004 17:21:49 -0400, Glen Daniels
> > <glen@thoughtcraft.com> wrote:
> > >
> > > OK, so dims, I'm going to -1 this change for this
> > release... can you
> > > back it out please?  I'm really sorry, but I think this is
> > too big a
> > > hit for our users to take.  If we really need to support
> > the SAAJ API all the way down through RPCElements, then we'll
> > have to do it by creating child MessageElements and keeping
> > them in sync with the RPCParams, at least for now....
> > >
> > > --Glen
> > >
> > >
> > >
> > > > -----Original Message-----
> > > > From: Eric.D.Friedman@wellsfargo.com
> > > > [mailto:Eric.D.Friedman@wellsfargo.com]
> > > > Sent: Thursday, July 08, 2004 12:19 PM
> > > > To: axis-dev@ws.apache.org; tomj@macromedia.com;
> > > > axis-tck@ws.apache.org
> > > > Cc: dims@apache.org
> > > > Subject: RE: Good to GO!!!
> > > >
> > > > Agreed, I've written code that works around the current
> > > > (incorrect) behavior.  This is one of those pay me now/
> > pay me later
> > > > choices -- either way, you have to pay.
> > > >
> > > > > -----Original Message-----
> > > > > From: Glen Daniels [mailto:gdaniels@sonicsoftware.com]
> > > > > Sent: Thursday, July 08, 2004 7:43 AM
> > > > > To: Tom Jordahl; axis-tck@ws.apache.org
> > > > > Cc: dims@apache.org; axis-dev@ws.apache.org
> > > > > Subject: RE: Good to GO!!!
> > > > >
> > > > >
> > > > >
> > > > > call.getOutputParams(), among other places.  Pretty
> > > > critical, I'd say.
> > > > >
> > > > > --Glen
> > > > >
> > > > > > -----Original Message-----
> > > > > > From: Tom Jordahl [mailto:tomj@macromedia.com]
> > > > > > Sent: Thursday, July 08, 2004 10:39 AM
> > > > > > To: 'axis-tck@ws.apache.org'; Glen Daniels
> > > > > > Cc: 'dims@apache.org'; 'axis-dev@ws.apache.org'
> > > > > > Subject: RE: Good to GO!!!
> > > > > >
> > > > > >
> > > > > > Do we think that the RPCParam API is public enough to
> > be used by
> > > > > > users?
> > > > > >
> > > > > > Is it in the list of classes we promised not too muck
> > with (do
> > > > > > we actually have that list? I know we wanted to have one)?
> > > > > >
> > > > > > Clearly our tests use it, but what is the use case
> > for Axis users?
> > > > > >
> > > > > > --
> > > > > > Tom Jordahl
> > > > > > Macromedia Server Development
> > > > > >
> > > > > > -----Original Message-----
> > > > > > From: Davanum Srinivas [mailto:davanum@gmail.com]
> > > > > > Sent: Thursday, July 08, 2004 10:33 AM
> > > > > > To: Glen Daniels
> > > > > > Cc: dims@apache.org; axis-tck@ws.apache.org;
> > > > axis-dev@ws.apache.org
> > > > > > Subject: Re: Good to GO!!!
> > > > > >
> > > > > > Glen,
> > > > > >
> > > > > > That's why i am making this change now rather than
> > later....if
> > > > > > we get lot of feedback from users, then we will have
> > to back it
> > > > > > out somehow.
> > > > > >
> > > > > > -- dims
> > > > > >
> > > > > > On Thu, 8 Jul 2004 10:30:21 -0400, Glen Daniels
> > > > > > <gdaniels@sonicsoftware.com> wrote:
> > > > > > >
> > > > > > > Sweet!  Thanks for the hard work dims!
> > > > > > >
> > > > > > > One question about the RPCParam change, though - if I'm
> > > > > reading it
> > > > > > > right, the getValue() call on RPCParam now returns the
text
> > > > > > value as
> > > > > > > per the SAAJ API, and the getObjectValue() is used to get
> > > > > > the actual
> > > > > > > value object.  This seems REALLY bad to me, in that there
> > > > > > might be a
> > > > > > > lot of preexisting code which will simply break when
> > > > this change
> > > > > > > occurs, and not in a very clean/easy-to-understand way
> > > > > either.  I'm
> > > > > > > pretty tempted to -1 this... what do others think?  If
> > > > > > we're going to
> > > > > > > make a change like this, it seems to me that we should
at
> > > > > > least give
> > > > > > > one release's worth of warning to developers.....
> > > > > > >
> > > > > > > --Glen
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > > -----Original Message-----
> > > > > > > > From: Davanum Srinivas [mailto:davanum@gmail.com]
> > > > > > > > Sent: Thursday, July 08, 2004 10:17 AM
> > > > > > > > To: axis-tck@ws.apache.org; axis-dev@ws.apache.org;
> > > > > Glen Daniels;
> > > > > > > > Glen Daniels
> > > > > > > > Subject: Good to GO!!!
> > > > > > > >
> > > > > > > > Glen,
> > > > > > > >
> > > > > > > > Ran both TCK's this morning....We're good to go for
the
> > > > > > beta release.
> > > > > > > >
> > > > > > > > Thanks,
> > > > > > > > dims
> > > > > > > > --
> > > > > > > > Davanum Srinivas - http://webservices.apache.org/~dims/
> > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > > --
> > > > > > Davanum Srinivas - http://webservices.apache.org/~dims/
> > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > >
> >
> >
> > --
> > Davanum Srinivas - http://webservices.apache.org/~dims/
> >
> >
> 


-- 
Davanum Srinivas - http://webservices.apache.org/~dims/

Mime
View raw message