Uploaded image for project: 'OFBiz'
  1. OFBiz
  2. OFBIZ-8337

Refactor and simplify the startup sequence in OFBiz

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 17.12.01
    • 17.12.01
    • base, start
    • None

    Description

      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.

      Attachments

        1. OFBIZ-8337-3.patch
          41 kB
          Taher Alkhateeb
        2. OFBIZ-8337-3.patch
          39 kB
          Taher Alkhateeb
        3. OFBIZ-8337-2.patch
          29 kB
          Taher Alkhateeb
        4. OFBIZ-8337.patch
          86 kB
          Taher Alkhateeb

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            taher Taher Alkhateeb
            taher Taher Alkhateeb
            Votes:
            1 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment