Uploaded image for project: 'Couchbase C client library libcouchbase'
  1. Couchbase C client library libcouchbase
  2. CCBC-1485

lcb_cntl LCB_CNTL_ENABLE_TRACING not disabling tracing

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 3.2.0
    • 3.2.2
    • library
    • None
    • 1

    Description

      From debugging session I found following behaviour.

      lcb_create creates a tracer per code in tracing.cc. 

      if (settings->use_tracing) {
      if (options && options->tracer)

      { settings->tracer = options->tracer; }

      else

      { settings->tracer = lcbtrace_new(obj, LCBTRACE_F_THRESHOLD); }

      }

      Given settings->use_tracing seems to always be 1 this is always executed.

      subsequent lcb_cntl call setting LCB_CNTL_ENABLE_TRACING to 0 just sets settings->use_tracing to 0 but does not touch settings->tracer which is tested for calling tracing code.

      Attachments

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

        Activity

          Ascending order - Click to sort in descending order
          jo JC added a comment -

          Follow up comment, first of all thanks for the fix, it works but I was wondering why this tracing feature is enabled by default ? Given it adds overhead on the client, should not it be disabled by default and only active if one calls lcb_cntl function ?

          Thanks

           

          jo JC added a comment - Follow up comment, first of all thanks for the fix, it works but I was wondering why this tracing feature is enabled by default ? Given it adds overhead on the client, should not it be disabled by default and only active if one calls lcb_cntl function ? Thanks  
          build-team Couchbase Build Team added a comment -

          Build couchbase-server-7.1.0-1450 contains libcouchbase commit c6c61f5 with commit message:
          CCBC-1485: allow to disable tracer for connected instance

          build-team Couchbase Build Team added a comment - Build couchbase-server-7.1.0-1450 contains libcouchbase commit c6c61f5 with commit message: CCBC-1485 : allow to disable tracer for connected instance

          People

            avsej Sergey Avseyev
            jo JC
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty