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

          perry Perry Krug created issue -
          daschl Michael Nitschinger made changes -
          Field Original Value New Value
          Assignee Raghavan Srinivas [ rags ] Michael Nitschinger [ daschl ]
          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?
          daschl Michael Nitschinger made changes -
          Fix Version/s 1.1beta [ 10370 ]
          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/)
          ingenthr Matt Ingenthron made changes -
          Fix Version/s 1.1.0 [ 10274 ]
          Fix Version/s 1.1-beta [ 10370 ]
          daschl Michael Nitschinger made changes -
          Fix Version/s 1.1.1 [ 10430 ]
          Fix Version/s 1.1.0 [ 10274 ]
          daschl Michael Nitschinger made changes -
          Link This issue duplicates JCBC-196 [ JCBC-196 ]
          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.
          daschl Michael Nitschinger made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Duplicate [ 3 ]
          ingenthr Matt Ingenthron made changes -
          Workflow jira [ 21622 ] Couchbase SDK Workflow [ 38427 ]

            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