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
              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
              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.

                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: