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

Don't execute OBSERVE protocol command if persistence and replication are 0

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.2 DP4
    • Fix Version/s: 1.2 Beta-2
    • Component/s: library
    • Labels:
      None

      Description

      Currently the behavior is to poll the servers in a loop, waiting until the observe timeout to fail the response. In the case of a configurable user application where e.g. a user may wish to sometimes disable persistence/replication, a use case may be to call the ExecuteStore function with Zero, Zero as arguments for persist and replicate.

      In this case, the internal observe implementation will wait a bit of time and then fail the command. Neither is desired behavior, as persistence is not expected and should be no-ops.

      [ There are probably other bugs lurking in this behavior which haven't been discovered yet.. ]

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

        Activity

        Hide
        mnunberg Mark Nunberg added a comment -

        This should be checked for both performance and reliability reasons.

        Show
        mnunberg Mark Nunberg added a comment - This should be checked for both performance and reliability reasons.
        Show
        mnunberg Mark Nunberg added a comment - http://review.couchbase.org/21088
        Hide
        mnunberg Mark Nunberg added a comment -
        Show
        mnunberg Mark Nunberg added a comment - http://review.couchbase.org/21088
        Hide
        john John Zablocki (Inactive) added a comment -

        Published and tagged

        Show
        john John Zablocki (Inactive) added a comment - Published and tagged

          People

          • Assignee:
            john John Zablocki (Inactive)
            Reporter:
            mnunberg Mark Nunberg
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Gerrit Reviews

              There are no open Gerrit changes