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 Fri, 09 Jul 2004 19:19:18 GMT
ABSOLUTELY +100000000000!!!!! :) Since that means that i don't have to
run the tcks again :) :)

-- dims



On Fri, 9 Jul 2004 15:16:52 -0400, Glen Daniels
<gdaniels@sonicsoftware.com> wrote:
> 
> I'm in the midst of investigating this a bit.
> 
> At this point, since it appears we don't really expose them any more, I
> think I'd be Ok with releasing the beta and noting this change IN BIG
> FLASHING LETTERS in the release notes so we can see if people have a
> problem with it.  If they do, we might have to do another beta in order
> to back it out.
> 
> Thoughts?
> 
> --Glen
> 
> > -----Original Message-----
> > From: Davanum Srinivas [mailto:davanum@gmail.com] 
> > Sent: Friday, July 09, 2004 3:15 PM
> > To:
> > Cc: dims@apache.org; Glen Daniels; axis-tck@ws.apache.org;
> > axis-dev@ws.apache.org; tomj@macromedia.com
> > Subject: Re: Good to GO!!!
> >
> > Jongjin,
> >
> > I need to be convinced that the current code in CVS breaks
> > backward compatibility...I don't see how Axis users are
> > working with RPCParam since it is an internal implementation
> > detail. Glen mentioned that
> > Call.getOutputParams() might be exposing this but a review of
> > the code shows that we don't expose RPCParam's there either...
> >
> > -- dims
> 
> 
> >
> >
> > On Sat, 10 Jul 2004 03:10:17 +0900, ? ?? <gunsnroz@hotmail.com> wrote:
> > > Hi all and Dims, Glen, Tom.
> > >
> > > The guys in Tmax Soft have another solution for this issue.
> > > Roughly, we modifed SerializationContextImpl, RPCElement, Call,
> > > RPCProvider and added a new class, DOM2EventRecorder.
> > > I think that our patch will keep the backward API compatibility.
> > > I'd like you to review our patch.
> > >
> > > I'll upload the patch to JIRA tomorrow because I need time to merge
> > > this patch to Axis nightly build.
> > > I hope our patch is acceptable to Axis.
> > >
> > > IMO, if JBoss and JonAS guys can submit anothoer patch, we
> > can choose
> > > the most sutiable one between them.
> > >
> > > /Jongjin.
> > > TmaxSoft Webservices Team
> > >
> > > >From: Davanum Srinivas <davanum@gmail.com>
> > > >Reply-To: dims@apache.org
> > > >To: Glen Daniels <gdaniels@sonicsoftware.com>
> > > >CC: axis-tck@ws.apache.org, dims@apache.org,
> > axis-dev@ws.apache.org,
> > > tomj@macromedia.com
> > > >Subject: Re: Good to GO!!!
> > > >Date: Thu, 8 Jul 2004 19:50:45 -0400
> > >
> > >
> > > >
> > > >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/
> > >
> > > _________________________________________________________________
> > > ??? ??? ??? ??? ??...  ?? MSN ?????? ?????!
> 
> 
> > > http://groups.msn.com/?pgmarket=ko-kr
> > >
> > >
> >
> >
> > --
> > Davanum Srinivas - http://webservices.apache.org/~dims/
> >
> >
> 


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

Mime
View raw message