Bugs item #763823, was opened at 2003-07-01 11:57
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=376685&aid=763823&group_id=22866

Category: JBossServer
Group: v3.2
Status: Open
Resolution: None
Priority: 5
Submitted By: Rod Burgett (rodburgett)
Assigned to: Nobody/Anonymous (nobody)
Summary: housekeeping in Main/SAR deployers

Initial Comment:
The purpose of this patch is to improve the cleanup 
operations of the deployers in org.jboss.deployment 
package, specifically the Main and SAR deployers.

The primary change to the MainDeployer is to 
supplement the shutdown method by nulling out 
references and clearing containers.

Also, error messages in the stop(DeploymentInfo) and 
destroy(DeploymentInfo) methods were updated to 
indicate which method produced the messsage.

Finally in the MainDeployer, the init(DeploymentInfo) 
method included a suspicious synchronized block.  The 
target code for synchronization adds an entry to the 
deploymentMap field, but the subject of the 
synchronized statement is a different field, 
deploymentList.  It is possible that this mismatch is 
intentional, but it suggests to me a cut-n-paste error.

The SARDeployer patch file adds a stopService() method 
to null out internal references that were initialized in 
startService().  Prior to updating these references, 
super.stopService() is called.


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=376685&aid=763823&group_id=22866


-------------------------------------------------------
This SF.Net email sponsored by: Free pre-built ASP.NET sites including
Data Reports, E-commerce, Portals, and Forums are available now.
Download today and enter to win an XBOX or Visual Studio .NET.
http://aspnet.click-url.com/go/psa00100006ave/direct;at.asp_061203_01/01
_______________________________________________
Jboss-development mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/jboss-development

Reply via email to