Uploaded image for project: 'Couchbase Go SDK'
  1. Couchbase Go SDK
  2. GOCBC-718

Some row parsing errors do not bubble to the user

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.0-beta, 2.0.0
    • Fix Version/s: 2.1.5
    • Component/s: library
    • Labels:
      None
    • Story Points:
      1
    • Sprint:
      SDK1: GA and Priority List

      Description

      The row parsing logic in SearchResult and ViewResult do not correctly bubble their parsing errors up to the user for handling. These should be fatal errors, as they indicate a corrupted response.

        Attachments

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

          Activity

          Show
          charles.dixon Charles Dixon added a comment - https://github.com/couchbase/gocb/blob/master/cluster_searchquery.go#L166
          Hide
          build-team Couchbase Build Team added a comment -

          Build couchbase-server-7.0.0-3115 contains gocb commit 894a45c with commit message:
          GOCBC-718: Expose search/view parsing errors through .Err

          Show
          build-team Couchbase Build Team added a comment - Build couchbase-server-7.0.0-3115 contains gocb commit 894a45c with commit message: GOCBC-718 : Expose search/view parsing errors through .Err
          Hide
          build-team Couchbase Build Team added a comment -

          Build sync_gateway-3.0.0-52 contains gocb commit 894a45c with commit message:
          GOCBC-718: Expose search/view parsing errors through .Err

          Show
          build-team Couchbase Build Team added a comment - Build sync_gateway-3.0.0-52 contains gocb commit 894a45c with commit message: GOCBC-718 : Expose search/view parsing errors through .Err

            People

            Assignee:
            charles.dixon Charles Dixon
            Reporter:
            brett19 Brett Lawson
            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