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

Operations are not retried if they hit an open circuit breaker

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 3.0.5
    • 3.2.0
    • library
    • None
    • 1
    • Critical

    Description

      I believe CircuitBreakerException should be retried.
      When the circuit breaker is Open, new requests throws CircuitBreakerException. They are not cached by the retry orchestrator and therefore not retried

      The retry orchestrator only catches CouchbaseExceptions and retries the ones that are marked with IRetryable. CircuitBreakerException should probably be caught by retry orchestrator and retried

      Attachments

        Issue Links

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

          Activity

            I found the underlying cause of this in our scenario.

            reported in NCBC-2664

            tommyja Tommy Jakobsen added a comment - I found the underlying cause of this in our scenario. reported in NCBC-2664
            jmorris Jeff Morris added a comment -

            The behavior is not correct currently; CircuitBreakerException should be retried until it times out.

            jmorris Jeff Morris added a comment - The behavior is not correct currently; CircuitBreakerException should be retried until it times out.
            jmorris Jeff Morris added a comment -

            Moving to 3.1.0 as it was not merged for 3.0.7.

            jmorris Jeff Morris added a comment - Moving to 3.1.0 as it was not merged for 3.0.7.
            jmorris Jeff Morris added a comment -

            Moving to 3.1.1 as it was not merged in 3.1.0!

            jmorris Jeff Morris added a comment - Moving to 3.1.1 as it was not merged in 3.1.0!
            jmorris Jeff Morris added a comment -

            This hasn't been merged yet. Moving to sdk 3.2.0

            jmorris Jeff Morris added a comment - This hasn't been merged yet. Moving to sdk 3.2.0

            People

              jmorris Jeff Morris
              tommyja Tommy Jakobsen
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty