Uploaded image for project: 'Couchbase Server'
  1. Couchbase Server
  2. MB-45384

Revisit the mutex and atomics usage for BleveDest and BleveDestPartition within pindex_bleve

Details

    • Task
    • Resolution: Unresolved
    • Major
    • Morpheus
    • Cheshire-Cat
    • fts
    • None
    • 1

    Description

      We've made changes that introduces both atomics and mutexes within the 2 structs - BleveDest and BleveDestPartition within cbft/pindex_bleve.go with these changes ..

      It's desirable to not use atomic.Value which makes the code less type-safe - so we should think of an alternative for ..

      • lastUUID within BleveDestPartition
      • partitions within BleveDest

      Also, the use of atomics and mutexes together makes the code more prone to error during development - so some work needed to improve things there as well.

      Attachments

        Issue Links

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

          Activity

            People

              likith.b Likith B
              abhinav Abhi Dangeti
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty