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

Codify and document DataStructure concurrency model

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 2.3.9, 2.4.0-dp2
    • Fix Version/s: 2.4.0
    • Component/s: docs, library
    • Labels:
      None

      Description

      We need to est. some expectations with respect to concurrency in the Couchbase.Collections namespace classes. These need to documented in the datastructures.dita and published.

        Attachments

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

          Activity

          Hide
          mike.goldsmith Michael Goldsmith added a comment -

          I've made this commit to update DataStructures method names and add a brief explanation around the use of CAS.
          https://github.com/couchbase/docs-cb4/commit/e26c61f9a6de50a4532cd50ef11c5cbcc2801069

          Jeff Morris Is this the type of thing you were expecting to be added?

          Show
          mike.goldsmith Michael Goldsmith added a comment - I've made this commit to update DataStructures method names and add a brief explanation around the use of CAS. https://github.com/couchbase/docs-cb4/commit/e26c61f9a6de50a4532cd50ef11c5cbcc2801069 Jeff Morris Is this the type of thing you were expecting to be added?
          Hide
          mike.goldsmith Michael Goldsmith added a comment -

          Please review what changes I've made to see if they cover what you expect.

          Show
          mike.goldsmith Michael Goldsmith added a comment - Please review what changes I've made to see if they cover what you expect.
          Hide
          jmorris Jeff Morris added a comment -

          I added a couple of comments. In general it looks great, but I wonder if we should document where (if we do) deviate from CAS?

          Show
          jmorris Jeff Morris added a comment - I added a couple of comments. In general it looks great, but I wonder if we should document where (if we do) deviate from CAS?
          Hide
          mike.goldsmith Michael Goldsmith added a comment -

          Added a few more commits to provide more info on when we use CAS in Data Structures:
          https://github.com/couchbase/docs-cb4/commits/feature/sdk-datastructures

          I did note that we use sync objects for some concurrency control, but that doesn't help in multi process situations. There may be more opportunities for CAS / Subdoc operations.

          Show
          mike.goldsmith Michael Goldsmith added a comment - Added a few more commits to provide more info on when we use CAS in Data Structures: https://github.com/couchbase/docs-cb4/commits/feature/sdk-datastructures I did note that we use sync objects for some concurrency control, but that doesn't help in multi process situations. There may be more opportunities for CAS / Subdoc operations.
          Hide
          jmorris Jeff Morris added a comment -

          Hi Michael Goldsmith - i assigned this to you so you can resolve when you are done.

          Show
          jmorris Jeff Morris added a comment - Hi Michael Goldsmith - i assigned this to you so you can resolve when you are done.
          Hide
          mike.goldsmith Michael Goldsmith added a comment -

          Documentation is updated, pending merge from the feature/sdk-datastructures branch into master

          Show
          mike.goldsmith Michael Goldsmith added a comment - Documentation is updated, pending merge from the feature/sdk-datastructures branch into master

            People

            • Assignee:
              mike.goldsmith Michael Goldsmith
              Reporter:
              jmorris Jeff Morris
            • 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

                  Error rendering 'com.pagerduty.jira-server-plugin:PagerDuty'. Please contact your Jira administrators.