[
https://issues.apache.org/jira/browse/OFBIZ-8337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15708700#comment-15708700
]
Jacopo Cappellato commented on OFBIZ-8337:
------------------------------------------
[~taher] thanks for your work.
I have reviewed your patch which looks great and also applied it to my local
environment: smoke testing was successful.
I suggest to commit it to trunk to perform some more tests and plan for the
next incremental steps.
One of these could be that of getting rid completely of the custom
NativeLibClassloader and remove the code that calls its addNativeClassPath(...)
method: it will be one step in the direction of having a simpler to deploy
system.
> Refactor and simplify the startup sequence in OFBiz
> ---------------------------------------------------
>
> Key: OFBIZ-8337
> URL: https://issues.apache.org/jira/browse/OFBIZ-8337
> Project: OFBiz
> Issue Type: Improvement
> Components: base, start
> Affects Versions: Upcoming Release
> Reporter: Taher Alkhateeb
> Assignee: Taher Alkhateeb
> Priority: Minor
> Attachments: OFBIZ-8337-2.patch, OFBIZ-8337.patch
>
>
> The startup sequence in OFBiz is highly complex and requires improvements on
> multiple levels including:
> - The entire classpath buildup logic and
> org.apache.ofbiz.base.start.Classpath needs to be removed. The original idea
> of classpath isolation between the components created many more problems than
> it solved, and right now the classpath construction is the responsibility of
> the build system.
> - The custom classloader needs to be removed as well together with the
> classpath mentioned above.
> - The StartupLoader interface should remove the start() method and just have
> two methods, load and unload.
> - The startup sequence should have only one StartupLoader, not an array of
> startup loaders. This StartupLoader (implemented as ContainerLoader) is the
> only class responsible for bootstrapping OFBiz
> - The ContainerLoader needs to be completely refactored, not only to remove
> the start() method but also to cleanup the very messy logic currently
> residing there.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
