The following issue has been updated:
Updater: Jacek Laskowski (mailto:[EMAIL PROTECTED])
Date: Sun, 14 Dec 2003 5:03 PM
Changes:
Attachment changed to patch.txt
---------------------------------------------------------------------
For a full history of the issue, see:
http://jira.codehaus.org/secure/ViewIssue.jspa?key=GERONIMO-131&page=history
---------------------------------------------------------------------
View the issue:
http://jira.codehaus.org/secure/ViewIssue.jspa?key=GERONIMO-131
Here is an overview of the issue:
---------------------------------------------------------------------
Key: GERONIMO-131
Summary: Remove nullification of DD's locations when the standard DD
doesn't exist
Type: Improvement
Status: Unassigned
Priority: Minor
Original Estimate: Unknown
Time Spent: Unknown
Remaining: Unknown
Project: Apache Geronimo
Components:
deployment
Assignee:
Reporter: Jacek Laskowski
Created: Sun, 14 Dec 2003 5:03 PM
Updated: Sun, 14 Dec 2003 5:03 PM
Description:
I spent the whole weekend to eventually find out that Geronimo deploys a
deployment unit only when it contains *two* required files: one that's mandated
by the appropriate spec and the other one - Geronimo-specific. So, when a EJB
bean is deployed two files must exist - /META-INF/ejb-jar.xml and
/META-INF/geronimo-ejb-jar.xml, otherwise it won't be deployed at all (and
unfortunatelly no message is printed out about it on the console). What caused
me to think about having only one? Just take a look at
org.openejb.nova.deployment.EJBModuleDeploymentPlanner that checks whether
geronimo-ejb-jar.xml exists and also a few lines below is the comment:
"currently everything is in the geronimo-ejb-jar.xml".
However, o.a.g.kernel.deployment.DeploymentHelper always returns null upon
being asked about standard or geronimo DD when the standard DD is not available
(see one of the constructor). Therefore, DeploymentHelper decides if a module
is deployable or not. I think it is not in charge of giving the answer. The
right of answering the question belongs to a planner, thus the change.
I also change a bit the class's javadoc.
---------------------------------------------------------------------
JIRA INFORMATION:
This message is automatically generated by JIRA.
If you think it was sent incorrectly contact one of the administrators:
http://jira.codehaus.org/secure/Administrators.jspa
If you want more information on JIRA, or have a bug to report see:
http://www.atlassian.com/software/jira
