On Feb 14, 2006, at 10:03 AM, Joe Bohn wrote:

David,

Thanks for the clarification. I'm still having some problems understanding our classloader construction ... so can I ask for a little more clarification?

When I encountered this problem it was because I had removed 3 derby dependencies (geronimo-derby, derby, and derbynet) from config j2ee-server. Daytrader had a parent of j2ee-server ... so it makes sense that this change could have affected daytrader. I had a fix that I hadn't made available yet where I added the dependencies directly into daytrader for these same three derby configs. However, I noticed that you chose instead to add system- database (which also included these 3 dependencies) as a parent of daytrader. What additional benefits does this provide?

I don't think your change would have worked, since I believe it would have still resulted in 2 copies of the derby classes being loaded and the derby engine (started and loaded from the system-database config) being in a different classloader than the jdbc classes trying to use it in the daytrader app.

Making daytrader a child of system-database forces all the classes to come from the system-database classloader.

Also, I'm working on additional updates. Would you prefer to have individual JIRAs for these items or larger ones with more "bang for buck" in reducing the size of minimal? What I have right now doesn't provide a lot of additional savings yet.

I think I might be happiest with a single jira item with mutliple patches on it: when we decide we can't get any smaller we can close it.

thanks
david jencks


Thanks,
Joe


David Jencks (JIRA) wrote:
[ http://issues.apache.org/jira/browse/GERONIMO-1613? page=comments#action_12366301 ] David Jencks commented on GERONIMO-1613:
----------------------------------------
To be a little clearer, I fixed the derby problems in daytrader also. r377628
Eliminate unncessary dependencies to reduce assemnbly footprint size
--------------------------------------------------------------------

        Key: GERONIMO-1613
        URL: http://issues.apache.org/jira/browse/GERONIMO-1613
    Project: Geronimo
       Type: Improvement
 Components: general
   Versions: 1.1
Environment: all
   Reporter: Joe Bohn
   Assignee: David Jencks
    Fix For: 1.1
Attachments: RemoveDeps.patch

Clean up assembly project.xml and eliminate some unnecessary dependencies in various modules and configs. This will reduce the footprint size (with special attention to the minimal-tomcat- assembly.
The patch contains the following:
- clean up minimal-tomcat-server\project.xml to remove commented out sections - clean up web-jms-tomcat-server\project.xml to remove commented out sections - remove dependencies from config\j2ee_server on xstream, jaxr- api, and geronimo-derby - remove dependencies from config\j2ee_deployer on geronimo- client-builder - remove dependencies from module\tomcat on activecluster, wadi- core, and wadi-tomcat55 There are still more dependencies that should be removed but this is a start. These changes reduce the disk footprint of minimal-tomcat-server from 27 meg to about 21 meg.

--
Joe Bohn
joe.bohn at earthlink.net

"He is no fool who gives what he cannot keep, to gain what he cannot lose." -- Jim Elliot

Reply via email to