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

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