-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Nick,

Just reviewed the code again...basically the AxisService is created inside the 
bowels of the deployer and by the time we
get it in the MR, it's too late. that's why we need the deprecated method...

thanks,
dims

~        Parameter endpointDescParam =
~                
service.getParameter(EndpointDescription.AXIS_SERVICE_PARAMETER);
~        if (endpointDescParam == null) {
~            ClassLoader serviceClassLoader = service.getClassLoader();
~            String serviceClass = (String) svcClassParam.getValue();
~            try {
~                Class clazz = Class.forName(serviceClass, true, 
service.getClassLoader());
~                ServiceDescription serviceDesc =
~                        
DescriptionFactory.createServiceDescriptionFromServiceImpl(
~                                clazz, service);
~            } catch (ClassNotFoundException e) {
~                throw new RuntimeException(
~                        
Messages.getMessage("JAXWSMessageReceiverNoServiceClass"));
~            }
~        }


Davanum Srinivas wrote:
| Nick,
|
| No, it does not reach the JAXWSDeployer in that case. Let me look at the
| other option. Since we were going to get rid of
| it, i did not spend much time.
|
| Would you agree that this is a lower priority task then the others?
|
| thanks,
| dims
|
|
| Nicholas L Gallardo wrote:
| | Dims,
| |
| | So will the new deployment code not run if someone has that pattern?
| i.e.,
| | the JAXWSDeployer will only work in the case where there's no
| srevices.xml
| | specified?
| |
| | What if we were to call the builder method that just takes a class and
| then
| | attach the AxisService after that?  Is that too late?
| |
| | I agree with Jeff in that I'd like to see it go.  I've struggled with the
| | different paths that exist in the building of ServiceDescription objects
| | and this would help clear some of that up.
| |
| | -Nick
| |
| |
| |
| |
| |
| |              Davanum Srinivas
| |              <[EMAIL PROTECTED]
| |
| m>                                                         To
| |                                        [email protected]
| |              03/20/2008
| 09:45                                           cc
| |              AM
| |
| Subject
| |                                        Re: [Axis2] Removing deprecated
| |              Please respond to         jaxws/metadata method
| |              [EMAIL PROTECTED]
| DescriptionFactory.createServiceDes
| |                    e.org               criptionFromServiceImpl(Class,
| |                                        AxisService)
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | Jeff,
| |
| | I'd prefer that we leave it in as deprecated for at least 1.4 release
| | before cleaning it up. It's used to support the
| | older 1.3's case of the JAXWS MR specified inside a services.xml.
| |
| | thanks,
| | dims
| |
| | Jeff Barrett wrote:
| | | All,
| | |
| | | A few of us have been discussing whether the deprecated method
| | |
| |
| 
org.apache.axis2.jaxws.description.DescriptionFactory.createServiceDescriptionFromServiceImpl
|
| | (Class,
| | | AxisService) can and/or should be removed.
| | |
| | | I think that it can and should be removed.  I has been deprecated for a
| | | long, long time, and with the changes that Dims and Roy did to
| integrate
| | | the JAXWS deployment into Axis2 recently, that method should no
| longer be
| | | used.  It goes down code paths that are very, very obsolete and the
| | method
| | | was only there to support the legacy testcases which have since been
| | moved
| | | to the new deployment model.
| | |
| | | With the integration into the deployement code, I think the code
| path in
| | | JAXWSMessageReceiver that is calling the deprecated method should be
| | | throwing an exception instead indicating that the endpoint was not
| | | correctly deployed.
| | |
| | | Thoughts?
| | |
| | | Thanks,
| | | Jeff
| | |
| | | IBM Software Group - WebSphere Web Services Development
| | | Phone: 512-838-4587 or Tie Line 678-4587
| | | Internet e-mail and Sametime ID: [EMAIL PROTECTED]
| | |
| | | ---------------------------------------------------------------------
| | | 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]
|
|
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (Cygwin)

iD8DBQFH4oS2gNg6eWEDv1kRArIkAKCi88Dowzhwlt9TJVrm0GfFW3+ybgCgifpW
zT+aoDu+bOT9SWfRz/0KedA=
=vDgN
-----END PGP SIGNATURE-----

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

Reply via email to