Uploaded image for project: 'Couchbase .NET client library'
  1. Couchbase .NET client library
  2. NCBC-3174

Out of Retries misclassified as Operation Timed Out

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Critical
    • 3.2.9
    • 3.2.8
    • None
    • None

    Description

      ExponentialBackoff sees that an operation has exceeded its max retries and throws `new OperationCancelledException`.

       

      RetryOrchestrator, meanwhile, catches this OperationCancelledException and assumes that it is a timeout.

       

      1) Rapid-fire failures should not be misclassified as timeouts

      2) SendQueueFullException should not be retryable, since this merely leads to rapid-fire timeout of nearly everything under heavy load.

      3) Timeout error messages should use the actual time taken to time out, not the time it was supposed to take to time out as that is misleading in cases like this and obfuscates cases where the actual time taken is much longer due to thread starvation.

      Attachments

        Issue Links

          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            People

              richard.ponton Richard Ponton
              richard.ponton Richard Ponton
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty