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

Handle xattr subdoc full document get/set

    XMLWordPrintable

Details

    Description

      Implement support for doing sub-document full-document get, add and sets for use with Xattrs.

      Attachments

        Issue Links

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

          Activity

            adamf Adam Fraser added a comment -

            Brett Lawson Will the sub-document full-document get/set/add payloads be run through the same transcoder used for single doc operations?  As discussed elsewhere, we're planning to use a custom transcoder to allow us to read/write documents as raw bytes, and would need that ability in the subdoc/full-doc operations.

            adamf Adam Fraser added a comment - Brett Lawson Will the sub-document full-document get/set/add payloads be run through the same transcoder used for single doc operations?  As discussed elsewhere, we're planning to use a custom transcoder to allow us to read/write documents as raw bytes, and would need that ability in the subdoc/full-doc operations.

            Hey Adam,

            The subdocument operations do not use the built in transcoders as they are guaranteed to be valid JSON (as that is the only thing SubDocument supports).  We are however adding the ability to directly read/write []byte's instead of being forced to marshal or unmarshal the data.

            brett19 Brett Lawson added a comment - Hey Adam, The subdocument operations do not use the built in transcoders as they are guaranteed to be valid JSON (as that is the only thing SubDocument supports).  We are however adding the ability to directly read/write []byte's instead of being forced to marshal or unmarshal the data.
            adamf Adam Fraser added a comment -

            That should be sufficient - thanks.

            adamf Adam Fraser added a comment - That should be sufficient - thanks.

            Build 5.0.0-2592 contains gocb commit df342b9a5303e91a7a706780ca01b79a5751e124 with commit message:
            GOCBC-181: Add support for Full-Doc Sub-Document operations.
            https://github.com/couchbase/gocb/commit/df342b9a5303e91a7a706780ca01b79a5751e124

            build-team Couchbase Build Team added a comment - Build 5.0.0-2592 contains gocb commit df342b9a5303e91a7a706780ca01b79a5751e124 with commit message: GOCBC-181 : Add support for Full-Doc Sub-Document operations. https://github.com/couchbase/gocb/commit/df342b9a5303e91a7a706780ca01b79a5751e124

            Build 5.0.0-2592 contains gocbcore commit cbfa27df3919808d0d3ba22ec3454ded33f048a3 with commit message:
            GOCBC-181: Add support for Full-Doc SubDocument operations.
            https://github.com/couchbase/gocbcore/commit/cbfa27df3919808d0d3ba22ec3454ded33f048a3

            build-team Couchbase Build Team added a comment - Build 5.0.0-2592 contains gocbcore commit cbfa27df3919808d0d3ba22ec3454ded33f048a3 with commit message: GOCBC-181 : Add support for Full-Doc SubDocument operations. https://github.com/couchbase/gocbcore/commit/cbfa27df3919808d0d3ba22ec3454ded33f048a3
            adamf Adam Fraser added a comment -

            After moving to 5.0.0-2682 to pick up MB-23522, there's no longer a way to set the ACCESS_DELETED flag when retrieving xattrs (now that it's been moved to the new subdoc_doc_flags).  

            This blocks uptake and testing of the rest of this functionality.

            adamf Adam Fraser added a comment - After moving to 5.0.0-2682 to pick up MB-23522 , there's no longer a way to set the ACCESS_DELETED flag when retrieving xattrs (now that it's been moved to the new subdoc_doc_flags).   This blocks uptake and testing of the rest of this functionality.
            brett19 Brett Lawson added a comment -

            Tracking the new bug under GOCBC-189.

            brett19 Brett Lawson added a comment - Tracking the new bug under GOCBC-189 .

            People

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

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty