ICEfaces
  1. ICEfaces
  2. ICE-8789

Make sleep configurable when ICEfaces ID is disposed when using Push Server/EPS

          Details

          • Type: Improvement Improvement
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: EE-1.8.2.GA_P04
          • Fix Version/s: EE-1.8.2.GA_P05
          • Component/s: Framework
          • Labels:
            None
          • Environment:
            ICEfaces, Push Server (EPS).
          • Affects:
            Documentation (User Guide, Ref. Guide, etc.), Compatibility/Configuration

            Description

            When an ICEfaces ID is disposed when using Push Server/EPS, an initial sleep of 1 second is done before actually disposing. In an environment where ICEfaces IDs are disposed often, the sleep is quite apparent.

              Activity

              Ascending order - Click to sort in descending order
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #32290 Thu Nov 22 08:01:18 MST 2012 jack.van.ooststroom Fixed JIRA ICE-8789 : Remove sleep when ICEfaces ID is disposed when using Push Server/EPS
              Files Changed
              Commit graph MODIFY /icefaces/trunk/icefaces/push-server/src/org/icefaces/push/server/SessionManager.java
              Migration created issue -
              Ken Fyten made changes -
              Field Original Value New Value
              Summary Placeholder issue Remove sleep when ICEfaces ID is disposed when using Push Server/EPS
              Ken Fyten made changes -
              Description Placeholder issue When an ICEfaces ID is disposed when using Push Server/EPS, an initial sleep of 1 second is done before actually disposing. In an environment where ICEfaces IDs are disposed often, the sleep is quite apparent.
              Ken Fyten made changes -
              Issue Type Bug [ 1 ] Improvement [ 4 ]
              Assignee Jack Van Ooststroom [ jack.van.ooststroom ]
              Security Private [ 10001 ]
              Fix Version/s EE-1.8.2.GA_P05 [ 10331 ]
              Reporter Migration [ remote ] Jack Van Ooststroom [ jack.van.ooststroom ]
              Affects Version/s EE-1.8.2.GA_P04 [ 10280 ]
              Environment Test ICEfaces, Push Server (EPS).
              Component/s Framework [ 10013 ]
              Hide
              Permalink
              Ken Fyten added a comment -

              Fixed JIRA ICE-8789 : Remove sleep when ICEfaces ID is disposed when using Push Server/EPS

              Show
              Ken Fyten added a comment - Fixed JIRA ICE-8789 : Remove sleep when ICEfaces ID is disposed when using Push Server/EPS
              Ken Fyten made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #32600 Tue Dec 04 09:43:40 MST 2012 jack.van.ooststroom Fixed JIRA ICE-8789 : Remove sleep when ICEfaces ID is disposed when using Push Server/EPS; Instead of removing the sleep, it's now configurable
              Files Changed
              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/PushServlet.java
              Ken Fyten made changes -
              Affects Documentation (User Guide, Ref. Guide, etc.),Compatibility/Configuration [ 10003, 10002 ]
              Jack Van Ooststroom made changes -
              Summary Remove sleep when ICEfaces ID is disposed when using Push Server/EPS Make sleep configurable when ICEfaces ID is disposed when using Push Server/EPS
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              The sleep(1000) was added a long time ago back when it was still called ICEfaces AHS. I tried to figure out why it was added, but I can't find the reason. I suspect it was being added for clustered environments. The sleep(1000) is now removed. We'll need to go through a full test cycle in order to determine the removal doesn't break any of the supported environments. If it does break a supported environment, we should come up with another solution. Marking this one as FIXED for now.

              Show
              Jack Van Ooststroom added a comment - The sleep(1000) was added a long time ago back when it was still called ICEfaces AHS. I tried to figure out why it was added, but I can't find the reason. I suspect it was being added for clustered environments. The sleep(1000) is now removed. We'll need to go through a full test cycle in order to determine the removal doesn't break any of the supported environments. If it does break a supported environment, we should come up with another solution. Marking this one as FIXED for now.
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              The sleep is now configurable. Removing it breaks WebSphere Application Server cluster fail-over support. Now the com.icesoft.faces.push.server.iceFacesIdDisposalDelay context parameter can be used to configure the sleep. In order to disable it it can be set to a long value <= 0. The default is the original 1000 ms.

              Show
              Jack Van Ooststroom added a comment - The sleep is now configurable. Removing it breaks WebSphere Application Server cluster fail-over support. Now the com.icesoft.faces.push.server.iceFacesIdDisposalDelay context parameter can be used to configure the sleep. In order to disable it it can be set to a long value <= 0. The default is the original 1000 ms.
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              The context parameter is intended for both Push Server and EPS.

              Show
              Jack Van Ooststroom added a comment - The context parameter is intended for both Push Server and EPS.
              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:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: