ICEpush
  1. ICEpush
  2. PUSH-374

Add automatic retry capability for failed push update retrieval requests

          Details

          • Type: Improvement Improvement
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: None
          • Fix Version/s: 4.1, EE-3.3.0.GA_P04
          • Component/s: JavaScript Client
          • Labels:
            None
          • Environment:
            ICEpush
          • Assignee Priority:
            P1
          • Affects:
            Documentation (User Guide, Ref. Guide, etc.)

            Description


            Although ICEpush supports automatic retries for failed heartbeat connections, we don’t do the same for failed push update retrieval attempts. Instead, we treat these the same as user-initiated requests that will immediately fail due to a network error.

            It would be friendlier if IcePush would handle failed push update retrievals by automatically retrying in a manner similar to that used for the existing heartbeat connections.

            The existing org.icepush.networkErrorRetryTimeouts configuration could be used for both heartbeat connection failures and push update retrieval failures.

            This change is especially important for push update retrieval requests which happen behind the scenes unbeknownst to the user. If I was a user, I might be confused by an asynchronous NCI popup that blocks further operations on my window due to a failed background ajax update which, if retried, would likely succeed.

              Activity

              Ascending order - Click to sort in descending order
              Ken Fyten created issue -
              Ken Fyten made changes -
              Field Original Value New Value
              Summary Add automatic retry capability for failed push update retrieval attempts Add automatic retry capability for failed push update retrieval requests
              Ken Fyten made changes -
              Assignee Mircea Toma [ mircea.toma ]
              Fix Version/s 4.1 [ 11570 ]
              Fix Version/s EE-3.3.0.GA_P04 [ 12271 ]
              Affects Documentation (User Guide, Ref. Guide, etc.) [ 10003 ]
              Assignee Priority P1 [ 10010 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #46110 Wed Oct 21 17:12:48 MDT 2015 mircea.toma PUSH-374 Implemented automatic retry capability for failed push update retrieval. Introduced org.icefaces.updateNetworkErrorRetryTimeouts context parameter to configure the retry intervals.
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/core/src/main/java/org/icefaces/util/EnvUtils.java
              Commit graph MODIFY /icefaces4/trunk/icefaces/core/src/main/javascript/application.js
              Commit graph MODIFY /icefaces4/trunk/icefaces/core/src/main/java/org/icefaces/impl/facelets/tag/icefaces/core/RefreshHandler.java
              Commit graph MODIFY /icefaces4/trunk/icefaces/core/src/main/java/org/icefaces/impl/event/BridgeSetup.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #46111 Wed Oct 21 17:33:57 MDT 2015 mircea.toma PUSH-374 Remove line used for debugging only.
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/core/src/main/javascript/application.js
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #46112 Wed Oct 21 17:37:06 MDT 2015 mircea.toma PUSH-374 Pass in 'retryIntervals' value.
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/core/src/main/javascript/application.js
              Hide
              Permalink
              Mircea Toma added a comment -

              Implemented automatic retry capability for failed push update retrieval. Also introduced org.icefaces.updateNetworkErrorRetryTimeouts context parameter to configure the intervals at which the bridge will retry to acquire the updates. The parameter value should be a list of space separated numbers which represent the retry intervals in milliseconds. The default value is '1000 2000 3000'.

              Show
              Mircea Toma added a comment - Implemented automatic retry capability for failed push update retrieval. Also introduced org.icefaces.updateNetworkErrorRetryTimeouts context parameter to configure the intervals at which the bridge will retry to acquire the updates. The parameter value should be a list of space separated numbers which represent the retry intervals in milliseconds. The default value is '1000 2000 3000' .
              Mircea Toma made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Mircea Toma
                  Reporter:
                  Ken Fyten
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: