loom
  1. loom
  2. LOOM-22

Break apart Environment.xml

    Details

    • Type: Improvement Improvement
    • Status: Open Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: future
    • Component/s: None
    • Labels:
      None
    • Number of attachments :
      0

      Description

      Currently we store all sorts of stuff in environment.xml like policy definition, classloader definition, logging definition etc. And in the future it is conceivable that we will be including things like interceptor definitions, management declarations etc.

      So what I was thinking is that we could break each aspect out into a separate file. At the moment that would mean files such as

      SAR-INF/logging.xml
      SAR-INF/classloaders.xml
      SAR-INF/policy.xml

      We could do it in a backwards compatabile fashion by first scanning environment.xml before looking for new config files. In most applications these configuration files can simply be ommitted and default values will be supplied (much like is done now). So no need to specify SAR-INF/classloaders.xml or SAR-INF/policy.xml unless you really want to define them.

      The advantages of this approach is that the configuration files become much much easier to validate using Schemas/DTDs as the config files are not mixed. It is also more future proof as we can simply add new config files as we need them if a new aspect warrants it.

      See thread: http://marc.theaimsgroup.com/?t=104651287700002&r=1&w=2

        Activity

        Peter Donald made changes -
        Field Original Value New Value
        Project JContainer [ 10131 ] loom [ 10210 ]
        Component/s Loom [ 10172 ]
        Key JCT-10 LOOM-22
        peter royal made changes -
        Fix Version/s 1.1 [ 10470 ]
        peter royal made changes -
        Fix Version/s 1.1 [ 10470 ]
        Fix Version/s future [ 10610 ]
        Johan Sjoberg made changes -
        Comment [ 28441 ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Mauro Talevi
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated: