Uploaded image for project: 'Couchbase Server'
  1. Couchbase Server
  2. MB-42430

ErrorMap indicates DurabilityImpossible is transient and should be retried

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 6.5.0, 6.5.1, 6.6.0
    • 7.0.0
    • memcached
    • None
    • Untriaged
    • 1
    • No

    Description

      Currently, the KV engine error map indicates that DurabilityImpossible is transient and should be retried, but I don't believe this is intended. It may be worthwhile to take a quick look at the other sync-durability error codes which were introduced to confirm they have the expected behaviour as well.

      Attachments

        Issue Links

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

          Activity

            owend Daniel Owen added a comment -

            Looked at install/etc/couchbase/kv/error_maps/error_map_v1.json

             "a1": {
                        "name": "DurabilityImpossible",
                        "desc": "Durability requirements are impossible to achieve",
                        "attrs": [
                            "item-only",
                            "retry-later"
                        ]
                    },
            

            Suggest should have the attribute of invalid-input

            owend Daniel Owen added a comment - Looked at install/etc/couchbase/kv/error_maps/error_map_v1.json "a1": { "name": "DurabilityImpossible", "desc": "Durability requirements are impossible to achieve", "attrs": [ "item-only", "retry-later" ] }, Suggest should have the attribute of invalid-input

            Build couchbase-server-7.0.0-3628 contains kv_engine commit c94b2d1 with commit message:
            MB-42430: DurabilityImpossible should be invalid-input

            build-team Couchbase Build Team added a comment - Build couchbase-server-7.0.0-3628 contains kv_engine commit c94b2d1 with commit message: MB-42430 : DurabilityImpossible should be invalid-input

            Closing this ticket based on unit_test run.

            ashwin.govindarajulu Ashwin Govindarajulu added a comment - Closing this ticket based on unit_test run.

            People

              brett19 Brett Lawson
              brett19 Brett Lawson
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty