Uploaded image for project: 'Couchbase PHP client library'
  1. Couchbase PHP client library
  2. PCBC-543

timeouts should not be logged by default

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.4.6
    • Fix Version/s: 2.6.2
    • Component/s: library
    • Labels:
      None

      Description

      When investigating tracing in PHP, I note that even though my exceptions have info, the timeouts log through PHP by default.

      [cb,WARN] (server L:505 I:-1833806363) <127.0.0.1:11210> (CTX=0x7ff334830190,memcached,SRV=0x7ff334953a00,IX=0) Failing command (pkt=0x7ff33580a000, opaque=4789, opcode=0x0) with error LCB_ETIMEDOUT (0x17)
      [cb,EROR] (server L:582 I:-1833806363) <127.0.0.1:11210> (CTX=0x7ff334830190,memcached,SRV=0x7ff334953a00,IX=0) Server timed out. Some commands have failed
      [cb,WARN] (server L:360 I:-1833806363) <127.0.0.1:11210> (CTX=0x7ff334830190,memcached,SRV=0x7ff334953a00,IX=0) Server sent us reply for a timed-out command. (OP=0x0, RC=0x0, SEQ=4789)
      

      The first one I think shouldn't be there at all. It should just be returned to the app. The second one seems to be the same as the first, but at error level. I know this is lcb generated and I saw roughly the same thing. The third is from tracing, and shouldn't be logged at WARN level from lcb. Maybe at DEBUG level, but otherwise it should only be in the threshold tracing logger.

        Attachments

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

          Activity

          Hide
          build-team Couchbase Build Team added a comment -

          Build couchbase-server-6.0.0-1153 contains libcouchbase commit a0c81d1 with commit message:
          PCBC-543: Update log levels for timed out operations

          Show
          build-team Couchbase Build Team added a comment - Build couchbase-server-6.0.0-1153 contains libcouchbase commit a0c81d1 with commit message: PCBC-543 : Update log levels for timed out operations
          Hide
          build-team Couchbase Build Team added a comment -

          Build couchbase-server-5.5.0-2759 contains libcouchbase commit a0c81d1 with commit message:
          PCBC-543: Update log levels for timed out operations

          Show
          build-team Couchbase Build Team added a comment - Build couchbase-server-5.5.0-2759 contains libcouchbase commit a0c81d1 with commit message: PCBC-543 : Update log levels for timed out operations
          Hide
          build-team Couchbase Build Team added a comment -

          Build libcouchbase-2.8.5-178 contains libcouchbase commit a0c81d1 with commit message:
          PCBC-543: Update log levels for timed out operations

          Show
          build-team Couchbase Build Team added a comment - Build libcouchbase-2.8.5-178 contains libcouchbase commit a0c81d1 with commit message: PCBC-543 : Update log levels for timed out operations
          Hide
          ingenthr Matt Ingenthron added a comment -

          Sergey Avseyev: looks like this is actually integrated/resolved. Can you close accordingly?

          Show
          ingenthr Matt Ingenthron added a comment - Sergey Avseyev : looks like this is actually integrated/resolved. Can you close accordingly?
          Hide
          avsej Sergey Avseyev added a comment -

          Yes, the timeouts logged by libcouchbase

          Show
          avsej Sergey Avseyev added a comment - Yes, the timeouts logged by libcouchbase

            People

            • Assignee:
              avsej Sergey Avseyev
              Reporter:
              ingenthr Matt Ingenthron
            • 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

                  Error rendering 'com.pagerduty.jira-server-plugin:PagerDuty'. Please contact your Jira administrators.