Uploaded image for project: 'UIMA'
  1. UIMA
  2. UIMA-838

Incorrect support for process CAS error threshold count and threshold window

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 2.2.2AS
    • Async Scaleout
    • None

    Description

      The thresholdAction should be taken when the number of CAS errors reaches the thresholdCount value within the last thresholdWindow CASes. A thresholdWindow of 0 means an infinite window, so the action is taken when the total number of errors reaches the thresholdCount. A thresholdCount of 0 means an infinite limit, errors are ignored. A thresholdCount of 1 means that the first error triggers the action, hence the action must be taken when the threshold is REACHED, not exceeded as stated in the current docs. (Although thresholds usually apply when exceeded, making the thresholdCount be the number permitted would require using a value such as -1 for no threshold, in contrast to our use of 0 to mean no timeout and no window)

      It should be an error to specify a (non-zero) thresholdWindow that is smaller than the thresholdCount ... an impossible situation.

      Note that if thresholdCount = 1 the thresholdWindow value is irrelevant, and if thresholdCount = thresholdWindow the action is taken when that many consecutive errors occur.

      Attachments

        1. UIMA-838.patch
          10 kB
          Burn L. Lewis
        2. UIMA-838-test.patch
          16 kB
          Burn L. Lewis
        3. UIMA-838-docs.patch
          7 kB
          Burn L. Lewis
        4. UIMA-838-docs2.patch
          2 kB
          Burn L. Lewis
        5. UIMA-838-fix2.patch
          1 kB
          Burn L. Lewis

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            schor Marshall Schor
            burn Burn L. Lewis
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment