[jboss-dev-forums] [Design the new POJO MicroContainer] - Re: Plague of temp VFS files and OOM on redeploy

[alesj]

"alesj" wrote : "scott.stark at jboss.org" wrote : 
  |   | The old MainDeployer has an explicit delete of existing content in the tmp directory when it was updated
  |   | 
  | OK, I'll check how we used to do this in old MD.
  | 
We did this in DeploymentInfo::cleanup, 
which was called in MD's undeploy --> destroy.

Should we have something similar in our DeployersImpl?

  |                unregisterMBean(context);
  |                removeClassLoader(context);
  |                cleanup(context); <-- HERE
  | 

Currently at HERE we only do metadata cleanup.
AbstractDeploymentContext:

  |    public void cleanup()
  |    {
  |       cleanupRepository(this);
  |    }
  | 
Perhaps adding something like this in AbstractVFSDeploymentContext

  |    @Override
  |    public void cleanup()
  |    {
  |       super.cleanup();
  |       root.close(); // or adding new VirtualFile::cleanup() method?
  |    }
  | 


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

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