[jboss-dev-forums] [Design of POJO Server] - Re: Fixing the WarClassLoaderDeployer

[scott.stark@jboss.org]

"adrian at jboss.org" wrote : The current way the WarClassLoaderDeployer works is all over the place. :-)
  | 
  | INCLUDE WEB-INF
  | 
  | I thought the requirement was to include the root of the war
  | since that is what the old UCL did?
  | Anyway this should be in the structure deployer not the classloader deployer.
  | 
No, this is the workaround for webservices packaging expectations of being able to load resources from the WEB-INF dir. Should be in WARStructure, but its isolated to the jbossas codebase for now.

"adrian at jboss.org" wrote : 
  | COPYING TO TEMP
  | 
  | Again this should be in the structure deployer. The VirtualFiles is adds
  | should be the temp ones.
  | 
this should not be needed.

"adrian at jboss.org" wrote : 
  | ALT-DD
  | 
  | This doesn't look like it is handled properly in the new deployers?
  | Shouldn't this be handled in the metadata parsing?
  | 
Yes, we are not handling this. Its a bit awkward as you would first have to scan the base descriptor for the alt path and then parse that file. Doesn't neatly fit into the current metadata parsing abstractions.

These issues are spread across the various war deployers, not just the WarClassLoaderDeployer. You wrote "is all of the place", but meant is all over the place. The war deployers and tomcat embedding code still await a major overhaul.


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

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