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

Configurable Get and Lock at EP Engine Level

    XMLWordPrintable

Details

    • Improvement
    • Resolution: Won't Fix
    • Major
    • feature-backlog
    • None
    • None
    • Security Level: Public
    • None

    Description

      This request came from the SDK team. As of 2.0 we did not want to make this capability public due to the complexity. Post-2.0 engineering agreed to look at this as a possible enhancement, and provide an easier way to configure maximum expiration and default expiration for get-and-lock operations on Server-side.

      > If no expiration time is set then the default expiration is 15 seconds. getl
      > also has a maximum expiration time of 29 seconds so any expiration time
      > greater than 29 seconds is automatically changed to 29 seconds.

      https://github.com/couchbase/ep-engine/blob/ece35893ac332077425d015e3b75b0f631abf4d5/ep_engine.cc#L494-L507

      The link above shows getLocked() function and here is the code snippet

      if (lockTimeout > max_timeout || lockTimeout < 1)

      { ... lockTimeout = default_timeout; }

      The snippet shows that:
      1) default and maximum values looks like configurable
      2) it will always reset to default value, for both unspecified and exceeding expiration time

      Therefore it would be nice to read here how to configure default and maximum timeout

      This note is also filed as disqus comment here (it could be removed after fix): http://www.couchbase.com/docs/couchbase-manual-2.0/couchbase-architecture-apis-memcached-protocol-additions.html#comment-541070264

      Attachments

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

        Activity

          People

            kzeller kzeller
            kzeller kzeller
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty