ICEfaces
  1. ICEfaces
  2. ICE-7596

Push Server: Improve locking around session data in the SessionManager

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-1.8.2.GA_P03
    • Fix Version/s: EE-1.8.2.GA_P04
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      Push Server, EPS

      Description

      The locking around the session data kept by the SessionManager in the Push Server needs to be improved as there is a window of opportunity to miss an update.

        Activity

        Repository Revision Date User Message
        ICEsoft Public SVN Repository #26981 Tue Dec 20 10:42:00 MST 2011 jack.van.ooststroom Fixed JIRA ICE-7596 : Push Server: Improve locking arround session data in the SessionManager
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/push-server/src/org/icefaces/push/server/Handler.java
        Commit graph ADD /icefaces/trunk/icefaces/push-server/src/org/icefaces/push/server/SessionContext.java
        Commit graph MODIFY /icefaces/trunk/icefaces/push-server/src/org/icefaces/push/server/SessionManager.java
        Commit graph MODIFY /icefaces/trunk/icefaces/push-server/src/org/icefaces/push/server/AbstractHandler.java
        Jack Van Ooststroom created issue -
        Jack Van Ooststroom made changes -
        Field Original Value New Value
        Assignee Jack van Ooststroom [ jack.van.ooststroom ]
        Hide
        Jack Van Ooststroom added a comment -

        Sending src/org/icefaces/push/server/AbstractHandler.java
        Sending src/org/icefaces/push/server/Handler.java
        Adding src/org/icefaces/push/server/SessionContext.java
        Sending src/org/icefaces/push/server/SessionManager.java
        Transmitting file data ....
        Committed revision 26981.

        Show
        Jack Van Ooststroom added a comment - Sending src/org/icefaces/push/server/AbstractHandler.java Sending src/org/icefaces/push/server/Handler.java Adding src/org/icefaces/push/server/SessionContext.java Sending src/org/icefaces/push/server/SessionManager.java Transmitting file data .... Committed revision 26981.
        Jack Van Ooststroom made changes -
        Salesforce Case []
        Hide
        Jack Van Ooststroom added a comment -

        Sending src/com/icesoft/faces/push/server/arp/internal/TomcatAdaptingServlet.java
        Sending src/com/icesoft/faces/push/server/arp/internal/WebLogicAdaptingServlet.java
        Transmitting file data ..
        Committed revision 29732.

        Show
        Jack Van Ooststroom added a comment - Sending src/com/icesoft/faces/push/server/arp/internal/TomcatAdaptingServlet.java Sending src/com/icesoft/faces/push/server/arp/internal/WebLogicAdaptingServlet.java Transmitting file data .. Committed revision 29732.
        Jack Van Ooststroom made changes -
        Salesforce Case []
        Jack Van Ooststroom made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Hide
        Jack Van Ooststroom added a comment -

        A new SessionContext class was introduced that amongst other objects contains a lock specific to the "session". This lock shared amongst the SessionManager and the WebLogicAdaptingServlet to ensure issues like skipping a beat can't occur anymore. Marking this one as FIXED.

        Show
        Jack Van Ooststroom added a comment - A new SessionContext class was introduced that amongst other objects contains a lock specific to the "session". This lock shared amongst the SessionManager and the WebLogicAdaptingServlet to ensure issues like skipping a beat can't occur anymore. Marking this one as FIXED.
        Jack Van Ooststroom made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Summary Push Server: Improve locking arround session data in the SessionManager Push Server: Improve locking around session data in the SessionManager
        Issue Type Bug [ 1 ] Improvement [ 4 ]
        Salesforce Case []
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: