Details

    • Type: Sub-task Sub-task
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.0
    • Component/s: ejb_container
    • Labels:
      None

      Description

      Please svn move them to ejb-container or a similar module.

      Thanks,
      Sahoo
      On Monday 12 December 2011 11:22 PM, Cheng Fang wrote:
      > Hi Sahoo,
      >
      > From the commit message, they were moved from ejb-container to ejb-connector to address issue http://java.net/jira/browse/GLASSFISH-17038 (3.1.1 deployment performance - ejb container module get loaded for a pure web application)
      >
      > Now that you've added @AnnotationHandlerFor to enable the dynamic handler loading, we can move them back to ejb-container.
      >
      > These ejb-related annotation handlers are now in ejb-connector module in both trunk and 3.1.2 workspaces. I noticed @AnnotationHandlerFor is only available in trunk. Sahoo, do you plan to port it to 3.1.2, too? If not, we will just leave ejb annotation handler classes in ejb-connector.
      >
      > r48934 | cf126330 | 2011-08-19 17:58:39 -0400 (Fri, 19 Aug 2011) | 2 lines
      >
      > Fix issue 17038 3.1.1 deployment performance - ejb container module get loaded for a pure web application. Move annotation handler
      > classes, archivist classes, scanner classes and related from ejb-container to ejb-connector, ran QL and ejb devtests. (the same s
      > et of fix is also in 3.1.2 branch.)
      >
      > -cheng
      >
      > On 12/9/11 10:27 PM, Sahoo wrote:
      >> Marina, Cheng,
      >>
      >> Do one of you remember why they are kept in connector module now? I would like them to be moved to some implementation module.
      >>
      >> Thanks,
      >> Sahoo
      >>
      >>

        Activity

        Hide
        Cheng Fang added a comment -

        committed to trunk only:
        r51608 | cf126330 | 2011-12-15 23:58:03 -0500 (Thu, 15 Dec 2011) | 5 lines

        Fix issue 17988 move ejb annotation handlers out of ejb-connector (in trunk only, opposite of r48934) now that handlers are dynamically loaded with @AnnotationHandlerFor.
        Move Import-Package: org.glassfish.apf.context entry from ejb-connector/osgi.bundle to ejb-container/osgi.bundle for AIX (see issue 10545)
        EjbAnnotationTypesProvider, EjbInWarArchivist and EjbInWarScanner remain under ejb-connector.
        Reviewed by Sahoo and dev@, ran QL and ejb devtests.

        Show
        Cheng Fang added a comment - committed to trunk only: r51608 | cf126330 | 2011-12-15 23:58:03 -0500 (Thu, 15 Dec 2011) | 5 lines Fix issue 17988 move ejb annotation handlers out of ejb-connector (in trunk only, opposite of r48934) now that handlers are dynamically loaded with @AnnotationHandlerFor. Move Import-Package: org.glassfish.apf.context entry from ejb-connector/osgi.bundle to ejb-container/osgi.bundle for AIX (see issue 10545) EjbAnnotationTypesProvider, EjbInWarArchivist and EjbInWarScanner remain under ejb-connector. Reviewed by Sahoo and dev@, ran QL and ejb devtests.

          People

          • Assignee:
            Cheng Fang
            Reporter:
            Sanjeeb Sahoo
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: