Uploaded image for project: 'Couchbase C client library libcouchbase'
  1. Couchbase C client library libcouchbase
  2. CCBC-1411

Further disambiguate 12009 error returned by Query into retriable-transient-errors and non-retriable-specific-errors

    XMLWordPrintable

Details

    • Improvement
    • Status: Open
    • Critical
    • Resolution: Unresolved
    • 3.1.3
    • 3.2.5
    • library
    • 1

    Description

      As noted in MB-45967, there is a need to further disambiguate 12009 error returned by Query into [1] retriable-transient-errors (NMVB, ENOMEM, ETMPFAIL etc) and [2] non-retriable-specific-errors (duplicate-key, key-not-found etc.) and set 'retriable' flag on lcb error code for [1]. Query Team mentioned 'cause' string returned with 12009 can be used for disambiguation. This will help applications/eventing decide whether to retry a failed operation or not.

      Attachments

        Issue Links

          For Gerrit Dashboard: CCBC-1411
          # Subject Branch Project Status CR V

          Activity

            jeelan.poola Jeelan Poola added a comment -

            Ray Cardillo Is this targeted to be resolved within Neo time frame? I ask as I do not know 3.3.0 timelines. Thank you!

            jeelan.poola Jeelan Poola added a comment - Ray Cardillo Is this targeted to be resolved within Neo time frame? I ask as I do not know 3.3.0 timelines. Thank you!
            jeelan.poola Jeelan Poola added a comment -

            Ray Cardillo Would be great if you can provide timelines for 3.3.0. We need to decide if the linked MB-43586 can be fixed in Neo timeframe or not. Thank you!

            jeelan.poola Jeelan Poola added a comment - Ray Cardillo Would be great if you can provide timelines for 3.3.0. We need to decide if the linked MB-43586 can be fixed in Neo timeframe or not. Thank you!
            ray.cardillo Ray Cardillo added a comment -

            Jeelan Poola we originally deferred this because we had other high priority work going on. That's still the case, but after review, I see there are still some outstanding questions we need to follow up on per Matt Ingenthron's previous reply. We can try to get some of that done and then see where things stand.

            I'll ping Michael Nitschinger to follow up on the specific question/topic Matt raised above and we'll update when we know more.

            ray.cardillo Ray Cardillo added a comment - Jeelan Poola we originally deferred this because we had other high priority work going on. That's still the case, but after review, I see there are still some outstanding questions we need to follow up on per Matt Ingenthron 's previous reply. We can try to get some of that done and then see where things stand. I'll ping Michael Nitschinger to follow up on the specific question/topic Matt raised above and we'll update when we know more.

            Jeelan Poola, I've created patch for libcouchbase, could you check how well it integrates with eventing and give feedback if you need something else: https://review.couchbase.org/c/libcouchbase/+/169468

            avsej Sergey Avseyev added a comment - Jeelan Poola , I've created patch for libcouchbase, could you check how well it integrates with eventing and give feedback if you need something else: https://review.couchbase.org/c/libcouchbase/+/169468

            Thank you Sergey Avseyev! Will revert asap.

            jeelan.poola Jeelan Poola added a comment - Thank you Sergey Avseyev ! Will revert asap.

            People

              abhishek.jindal Abhishek Jindal
              jeelan.poola Jeelan Poola
              Votes:
              0 Vote for this issue
              Watchers:
              9 Start watching this issue

              Dates

                Created:
                Updated:

                Gerrit Reviews

                  There is 1 open Gerrit change

                  PagerDuty