ICEpush
  1. ICEpush
  2. PUSH-178

Improve control of annotation scanning performance (Part 1)

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.0.1, EE 3.0.0
    • Fix Version/s: 3.1, EE-3.0.0.GA_P01
    • Component/s: Push Library
    • Labels:
      None
    • Environment:
      ICEpush
    • Assignee Priority:
      P1
    • Affects:
      Documentation (User Guide, Ref. Guide, etc.), Compatibility/Configuration

      Description

      The PushGroupManagerFactory.newPushGroupManager() method has some code that attempts to scan the classpath for implementations of an interface to determine which PushGroupManager implementation to use. The main use case for this is detecting EPS. In applications or environments where the classpath contains many libraries, this can cause a performance issue during the initial request.

      We discussed two strategies for dealing with this. For the short-term, we should supply a context-parameter that allows this to be turned off for environments that know EPS is not being used. In the long-term, we should switch to using the TLD to simply specify which implementation should be used.

        Activity

        Repository Revision Date User Message
        ICEsoft Public SVN Repository #29922 Wed Jul 11 07:41:46 MDT 2012 jack.van.ooststroom Updating icepush.jar after fix to PUSH-178
        Files Changed
        Commit graph MODIFY /icefaces3/trunk/icefaces/lib/icepush.jar
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #29921 Wed Jul 11 07:40:44 MDT 2012 jack.van.ooststroom Fixed JIRA PUSH-178 : Improve control of annotation scanning performance (Part 1)
        Files Changed
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/PushGroupManagerFactory.java
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/servlet/MainServlet.java

          People

          • Assignee:
            Jack Van Ooststroom
            Reporter:
            Deryk Sinotte
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: