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

ThresholdLoggingTracer should include timeout

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 3.1.7
    • 3.2.0
    • None
    • 1

    Description

      On master the JSON output of the threshold logging tracer does not include the timeout_ms attribute as a number, the RFC mandates it though.

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

          Appears to missing from Java as well: https://github.com/couchbase/couchbase-jvm-clients/blob/master/core-io/src/main/java/com/couchbase/client/core/cnc/tracing/ThresholdLoggingTracer.java#L54-L64

          jmorris Jeff Morris added a comment - Appears to missing from Java as well: https://github.com/couchbase/couchbase-jvm-clients/blob/master/core-io/src/main/java/com/couchbase/client/core/cnc/tracing/ThresholdLoggingTracer.java#L54-L64
          daschl Michael Nitschinger added a comment -

          Jeff Morris yes good point. Actually I double checked the RFC and it was missing there too on the threshold logging tracer, but it should be there anyways. I'm going to add it to java and add it to the rfc.. it's a small but useful addition when debugging I think

          daschl Michael Nitschinger added a comment - Jeff Morris yes good point. Actually I double checked the RFC and it was missing there too on the threshold logging tracer, but it should be there anyways. I'm going to add it to java and add it to the rfc.. it's a small but useful addition when debugging I think

          People

            jmorris Jeff Morris
            daschl Michael Nitschinger
            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