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

Ensure N1QL queries are retried after evicting invalid prepared statement

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.5.2
    • Component/s: None
    • Labels:
      None

      Description

      Caused by this change: https://github.com/couchbase/couchbase-net-client/commit/55f05bb65a075766fb5493386315178b6a4840cb

      CouchbaseRequestExecutor.RetryQueryEveryAsync retry behaviour does not use QueryClient.CheckRetry where query retry logic is located.

        Attachments

          Issue Links

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

            Activity

            Hide
            mike.goldsmith Michael Goldsmith added a comment -

            My mistake in re-adding CheckRetry and retrying directly in QueryClient. The intent was to use CouchbaseRequestExecutor for retry behaviour.

            Show
            mike.goldsmith Michael Goldsmith added a comment - My mistake in re-adding CheckRetry and retrying directly in QueryClient. The intent was to use CouchbaseRequestExecutor for retry behaviour.
            Hide
            mike.goldsmith Michael Goldsmith added a comment -

            After discussing with Jeff, we're going revert change NCBC-1502 for 2.5.1 and move this ticket to 2.5.2 to ensure there are no regressions in moving the retry behaviour to the request executor level.

            Show
            mike.goldsmith Michael Goldsmith added a comment - After discussing with Jeff, we're going revert change NCBC-1502  for 2.5.1 and move this ticket to 2.5.2 to ensure there are no regressions in moving the retry behaviour to the request executor level.

              People

              • Assignee:
                mike.goldsmith Michael Goldsmith
                Reporter:
                mike.goldsmith Michael Goldsmith
              • 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

                    Error rendering 'com.pagerduty.jira-server-plugin:PagerDuty'. Please contact your Jira administrators.