glassfish
  1. glassfish
  2. GLASSFISH-20379

Provide switch for jersey eager loading on/off

    Details

    • Type: Improvement Improvement
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 4.0_b85
    • Fix Version/s: 4.1
    • Component/s: admin
    • Labels:
      None

      Description

      There are some issues with parallel Jersey eager loading in some environments.
      Sahoo ask for switch on/off this feature based on domain.xml value.

        Activity

        Hide
        martin.mares added a comment -

        SVN 61838: Temporary switch off eager loading of admin framework. - Lazy load: First CLI call triggers init.

        Show
        martin.mares added a comment - SVN 61838: Temporary switch off eager loading of admin framework. - Lazy load: First CLI call triggers init.
        Hide
        martin.mares added a comment -

        Tom recommends:
        1. temporarily disable eager loading until the concurrency problem is fixed.
        2. continue working with the Jersey and integration teams to get the
        concurrency problem fixed.
        3. design a different method for triggering eager loading. For example,
        one method might be to implement an @Async remote command that will
        cause the command framework to be initialized (and Jersey loaded). The
        start-domain command could then be modified to invoke that command after
        the server has started (since we always want that initialization to
        happen if a DAS is started with start-domain). This will allow embedded
        environment to optionally choose if they want this to happen.

        Show
        martin.mares added a comment - Tom recommends: 1. temporarily disable eager loading until the concurrency problem is fixed. 2. continue working with the Jersey and integration teams to get the concurrency problem fixed. 3. design a different method for triggering eager loading. For example, one method might be to implement an @Async remote command that will cause the command framework to be initialized (and Jersey loaded). The start-domain command could then be modified to invoke that command after the server has started (since we always want that initialization to happen if a DAS is started with start-domain). This will allow embedded environment to optionally choose if they want this to happen.
        Hide
        Sanjeeb Sahoo added a comment -

        Reassigning this to Martin Mare. Every use of GlassFish does not require Jersey (e.g., embedded ejb container or embedde web container, etc.), so I strongly recommend to provide an option to disable initialization of a component unnecessarily. Let's not add to footprint to start fast.

        Current state of things are many people including QA is spending time because of bug #16731202. We need to provide a work around so that teams can effectively carry out their tasks.

        Show
        Sanjeeb Sahoo added a comment - Reassigning this to Martin Mare. Every use of GlassFish does not require Jersey (e.g., embedded ejb container or embedde web container, etc.), so I strongly recommend to provide an option to disable initialization of a component unnecessarily. Let's not add to footprint to start fast. Current state of things are many people including QA is spending time because of bug #16731202. We need to provide a work around so that teams can effectively carry out their tasks.
        Hide
        martin.mares added a comment -

        Hi Sahoo,
        As you see in upper comments it will be nice if you or John can mail (or comment) here more details about the core problem for Tom.
        Thanks.

        Show
        martin.mares added a comment - Hi Sahoo, As you see in upper comments it will be nice if you or John can mail (or comment) here more details about the core problem for Tom. Thanks.
        Hide
        michael.y.chen added a comment -

        Tom, Martin, Sahoo, please figure out if this is needed for 4.0.1.

        Show
        michael.y.chen added a comment - Tom, Martin, Sahoo, please figure out if this is needed for 4.0.1.

          People

          • Assignee:
            martin.mares
            Reporter:
            martin.mares
          • Votes:
            1 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated: