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

ifdef'd todo in subdoc code from 2015

    XMLWordPrintable

Details

    • Task
    • Status: Closed
    • Major
    • Resolution: Fixed
    • .master
    • 7.1.0
    • memcached

    Attachments

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

      Activity

        Build couchbase-server-7.1.0-2209 contains kv_engine commit 422c4ae with commit message:
        MB-25813: Fixup test case and remove @todo

        build-team Couchbase Build Team added a comment - Build couchbase-server-7.1.0-2209 contains kv_engine commit 422c4ae with commit message: MB-25813 : Fixup test case and remove @todo
        drigby Dave Rigby added a comment - - edited

        So there's no behaviour change in the server in Neo - attempting to insert a non-primitive type via ArrayAddUnique returns status code SubdocValueCantinsert; just as it did in prior versions.

        If the SDK is making changes to handle this scenario in a different way - it sounds like they are now checking for the argument type client-side and not sending it to the server if not primitive - then that is a behaviour change in the SDK and hence should be release-noted in the appropriate SDK release(s).

        drigby Dave Rigby added a comment - - edited So there's no behaviour change in the server in Neo - attempting to insert a non-primitive type via ArrayAddUnique returns status code SubdocValueCantinsert; just as it did in prior versions. If the SDK is making changes to handle this scenario in a different way - it sounds like they are now checking for the argument type client-side and not sending it to the server if not primitive - then that is a behaviour change in the SDK and hence should be release-noted in the appropriate SDK release(s).

        Thanks Dave Rigby that's helpful context, I may have misunderstood the ask from SDK team here, will refer back to them!

        hakim.cassimally Hakim Cassimally added a comment - Thanks Dave Rigby that's helpful context, I may have misunderstood the ask from SDK team here, will refer back to them!

        People

          trond Trond Norbye
          will.gardner Will Gardner (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