You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@geronimo.apache.org by ji...@codehaus.org on 2003/12/15 00:04:56 UTC

[jira] Created: (GERONIMO-131) Remove nullification of DD's locations when the standard DD doesn't exist

Message:

  A new issue has been created in JIRA.

---------------------------------------------------------------------
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