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

Codify and document DataStructure concurrency model

    XMLWordPrintable

Details

    • Improvement
    • Status: Resolved
    • Critical
    • Resolution: Fixed
    • 2.3.9, 2.4.0-dp2
    • 2.4.0
    • docs, library
    • 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

          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?

          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?

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

          mike.goldsmith Michael Goldsmith added a comment - Please review what changes I've made to see if they cover what you expect.
          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?

          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?

          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.

          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.
          jmorris Jeff Morris added a comment -

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

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

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

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

          People

            mike.goldsmith Michael Goldsmith
            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