Uploaded image for project: 'Couchbase .NET client library'
  1. Couchbase .NET client library
  2. NCBC-2639

Upsert-and-remove doesn't work

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Incomplete
    • None
    • 3.0.6
    • None
    • None
    • 1

    Description

      https://hackmd.io/Eaf20XhtRhi8aGEn_xIH8A?view#Upsert-and-remove

      Upsert-and-remove pattern is required for transactions support, but is currently not producing the correct results.

      Attachments

        Issue Links

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

          Activity

            jmorris Jeff Morris added a comment -

            Richard Ponton - can this be closed?

            jmorris Jeff Morris added a comment - Richard Ponton - can this be closed?

            Jeff Morris, Nope. Still open.

              .Upsert(TransactionFields.TransactionInterfacePrefixOnly, (string?)null, isXattr: true)
              .Remove(TransactionFields.TransactionInterfacePrefixOnly, isXattr: true)
              .SetDoc(finalDoc),
            

            This fails with an InvalidPacket error.

            richard.ponton Richard Ponton added a comment - Jeff Morris , Nope. Still open. .Upsert(TransactionFields.TransactionInterfacePrefixOnly, ( string ?) null , isXattr: true ) .Remove(TransactionFields.TransactionInterfacePrefixOnly, isXattr: true ) .SetDoc(finalDoc), This fails with an InvalidPacket error.

            Turns out it's Upsert("field", null, isXattr: true) that doesn't work.

            Upsert-and-remove works fine with Upsert("field", string.Empty, isXattr: true)

            richard.ponton Richard Ponton added a comment - Turns out it's Upsert("field", null, isXattr: true) that doesn't work. Upsert-and-remove works fine with Upsert("field", string.Empty, isXattr: true)

            People

              richard.ponton Richard Ponton
              richard.ponton Richard Ponton
              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