Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Duplicate
    • Affects Version/s: 1.0.3
    • Fix Version/s: 1.1.1
    • Component/s: Documentation
    • Security Level: Public
    • Labels:
      None

      Description

      Hopefully this is just a doc bug and not in the actual code, but all of the atomic updates including prepend (http://www.couchbase.com/docs/couchbase-sdk-java-1.0/couchbase-sdk-java-update-prepend.html) should not require CAS. That significantly defeats their performance effectiveness.

      If the function does require CAS, we should document that a CAS id of 0 will override the requirement to get the key again as in this example:
      client.prepend(casv.getCas(),"samplekey", "prependedstring");

        Issue Links

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

          Activity

          Hide
          daschl Michael Nitschinger added a comment -

          This is a duplicate for JCBC-196, see the process over there.

          Show
          daschl Michael Nitschinger added a comment - This is a duplicate for JCBC-196 , see the process over there.
          Hide
          perry Perry Krug added a comment -

          I think a simple doc enhancement to inform the user that they can replace the actual CAS call with a 0 to avoid the second round-trip to the server. There are a few situations where CAS would be appropriate for these ops, so it doesn't make sense to remove it from the library, but the most common case is to use them as atomic operations and therefore not need CAS...so the doc update should just make that clearer. Please make sure to note it on the various other atomic ops (incr/dev/append/touch/)

          Show
          perry Perry Krug added a comment - I think a simple doc enhancement to inform the user that they can replace the actual CAS call with a 0 to avoid the second round-trip to the server. There are a few situations where CAS would be appropriate for these ops, so it doesn't make sense to remove it from the library, but the most common case is to use them as atomic operations and therefore not need CAS...so the doc update should just make that clearer. Please make sure to note it on the various other atomic ops (incr/dev/append/touch/)
          Hide
          daschl Michael Nitschinger added a comment -

          This is not a doc bug, both prepend and append require CAS values.

          So is this still a doc enhancement or should we consider enhancing the library?

          Show
          daschl Michael Nitschinger added a comment - This is not a doc bug, both prepend and append require CAS values. So is this still a doc enhancement or should we consider enhancing the library?

            People

            • Assignee:
              daschl Michael Nitschinger
              Reporter:
              perry Perry Krug
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Gerrit Reviews

                There are no open Gerrit changes