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

GoCB appears to be ignoring partial view query errors

    XMLWordPrintable

    Details

      Description

      In certain cases, couchbase server views can return a 200 response code with partial errors embedded.  See this ngrep capture for example HTTP traffic.

      Based on looking at the GoCB code, I'm seeing a mismatch in field names related to partial errors:

       

      Depending on the outcome of the discussion in MB-25068 (see comment about handling failure cases), it might also be useful to add the ability of the caller to get at these partial errors.

      In the Sync Gateway case, since we are essentially proxying view query results back to a client that is calling the SG Views API, we just need a way to propagate these errors back.  At the moment, it doesn't look like GoCB is exposing the errors that can come back from Couchbase Server in the "errors" field of the view query response.

       

      Versions

      Gocb commit: 0cd63ba8b594091ea0005ec50ee21299e0b22d97

      Gocb core commit: 

      6b3fcad8ac5e2212fad55e57d481cb056b99f3c0

       

      Related tickets

       

      https://issues.couchbase.com/browse/MB-25068

      https://github.com/couchbase/sync_gateway/issues/2682

        Attachments

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

          Activity

          Show
          traun Traun Leyden (Inactive) added a comment - Related go-couchbase code:   https://github.com/couchbase/go-couchbase/blob/master/views.go#L24-L41
          Hide
          daniel.petersen Daniel Petersen added a comment -

          Any idea when this will be finished? Sync Gateway team is trying to close out all issues by this Friday (August 11)?

          Show
          daniel.petersen Daniel Petersen added a comment - Any idea when this will be finished? Sync Gateway team is trying to close out all issues by this Friday (August 11)?
          Hide
          build-team Couchbase Build Team added a comment -

          Build 5.1.0-1448 contains gocb commit 6382451e5ef472cd19e4eba6e82e3c1957e4cc09 with commit message:
          GOCBC-224: Return partial view errors during Close call.
          https://github.com/couchbase/gocb/commit/6382451e5ef472cd19e4eba6e82e3c1957e4cc09

          Show
          build-team Couchbase Build Team added a comment - Build 5.1.0-1448 contains gocb commit 6382451e5ef472cd19e4eba6e82e3c1957e4cc09 with commit message: GOCBC-224 : Return partial view errors during Close call. https://github.com/couchbase/gocb/commit/6382451e5ef472cd19e4eba6e82e3c1957e4cc09
          Hide
          build-team Couchbase Build Team added a comment -

          Build 5.0.1-4738 contains gocb commit 6382451e5ef472cd19e4eba6e82e3c1957e4cc09 with commit message:
          GOCBC-224: Return partial view errors during Close call.
          https://github.com/couchbase/gocb/commit/6382451e5ef472cd19e4eba6e82e3c1957e4cc09

          Show
          build-team Couchbase Build Team added a comment - Build 5.0.1-4738 contains gocb commit 6382451e5ef472cd19e4eba6e82e3c1957e4cc09 with commit message: GOCBC-224 : Return partial view errors during Close call. https://github.com/couchbase/gocb/commit/6382451e5ef472cd19e4eba6e82e3c1957e4cc09

            People

            • Assignee:
              brett19 Brett Lawson
              Reporter:
              traun Traun Leyden (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              4 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.