Hi Deepal, Adding custom deployers is indeed a must. I would like to point out several other things, which are currently not possible with the deployer:
- remote deployment is necessary (currently it is not possible) - hot deployment is good in words, but in reality it doesn't help when one wants to receive any proper feedback immediately after deployment about the state of the service. - Users should be able to have both "hot-deployment" (by dropping files in a dir), and remote deployment (i.e a servlet that calls doDeploy() and doUndeploy). - I took a look at your interface and i have the following objection: It still doesn't propagate exceptions up to the caller. This is very important. the DeploymentException should be thrown up to the caller so that he can react to that, especially in the case of remote deployment. It doesn't support undeployment as far as i can see from a first look - is that planned to be added or is it located in some other code? As for the remote deployment, Axis2 should have a plain Servlet that manages remote requests, and a client deployment API. I may contribute that actually. This is very important when it comes to enterprise set-ups. The current AdminServlet is not suitable for calling remotely via HTTP, because it renders JSP pages -> HTML , and it is not elegant to parse HTML... There should be a RemoteDeployServlet that supports deploy/undeploy/ , etc. I have some other ideas which can be implemented in the future, such as transactional deployment for service groups, versioning of deployed archives, dependencies among deployed archives, and so on. This will add a great value to the stack. Thanks very much, I think we can coordinate the deployment discussion in some way and I can contribute as well to that. Best, Angel On 2/9/07, Deepal Jayasinghe <[EMAIL PROTECTED]> wrote:
Hi Thilina ; I have committed the changes ; please have a look at org.apache.axis2.deployment.Deployer and there are two impl as well , one for handling service and other for modules. Thanks Deepal > I'm definitely +1 for this.. In fact I started doing the same couple > of days ago :).. > > BTW it would be great if you can provide more details of the interface > which you are going to provide.. > > Thanks, > Thilina > --------------------------------------------------------------------- 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]