[jboss-dev-forums] [Design the new POJO MicroContainer] - Re: Redeploy is done wrong

[adrian@jboss.org]

"alesj" wrote : 
  | "alesj" wrote : 
  |   | This would mean we have to move all the maps that track failing deployments
  |   | from MainDeployerImpl into DeployersImpl.
  |   | 
  |   | How does this relate with method signature changes?
  |   | 
  | This means I have to move everything (map, comparators, ...) to DeployersImpl.
  | As there is no point in hacking again, to have some ugly callbacks to MainDeployersImpl.
  | 
  | I'll keep MainDeployerImpls's interfaces, which will only delegate to DeployersImpl, OK?
  | 
  | I guess this means we'll have to do Deployers 2.1?

Why can't you just change MainDeployersImpl.process() to be?


  | Deployers.process(toUndeploy, Collections.emptyList());
  | determineStructureForToDeploy();
  | Deployers.process(Collection.emptyList(), toDeploy);
  | 

That doesn't involve any changes to the external api.

The thing that changes is that you can't get the DeploymentContext/Unit from the
MainDeployer until you've invoked process() whereas previously
you could get it after addDeployment().

View the original post : http://www.jboss.org/index.html?module=bb&op=viewtopic&p=4210657#4210657

Reply to the post : http://www.jboss.org/index.html?module=bb&op=posting&mode=reply&p=4210657