Details

    • Type: Bug
    • Status: Open
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: 1.1.9
    • Fix Version/s: .backlog1.x
    • Component/s: Core
    • Security Level: Public
    • Labels:
      None

      Description

      Not blocking anyone, but the library could be more resilient by properly handling a null transcoder instead of throwing the following exception:

      Aug 27 09:40:07: INFO: Reconnecting due to exception on

      {QA sa=localhost/127.0.0.1:11210, #Rops=1, #Wops=0, #iq=0, topRop=Cmd: -108 Opaque: 41 Key: 10 Exp: 10, topWop=null, toWrite=0, interested=1}

      Aug 27 09:40:07: java.lang.NullPointerException
      Aug 27 09:40:07: at com.couchbase.client.CouchbaseClient$9.gotData(CouchbaseClient.java:953)
      Aug 27 09:40:07: at net.spy.memcached.protocol.binary.GetlOperationImpl.decodePayload(GetlOperationImpl.java:59)
      Aug 27 09:40:07: at net.spy.memcached.protocol.binary.OperationImpl.finishedPayload(OperationImpl.java:178)
      Aug 27 09:40:07: at net.spy.memcached.protocol.binary.OperationImpl.readFromBuffer(OperationImpl.java:162)
      Aug 27 09:40:07: at net.spy.memcached.protocol.binary.GetlOperationImpl.readFromBuffer(GetlOperationImpl.java:31)
      Aug 27 09:40:07: at net.spy.memcached.MemcachedConnection.handleReads(MemcachedConnection.java:563)
      Aug 27 09:40:07: at net.spy.memcached.MemcachedConnection.handleIO(MemcachedConnection.java:480)
      Aug 27 09:40:07: at net.spy.memcached.MemcachedConnection.handleIO(MemcachedConnection.java:261)
      Aug 27 09:40:07: at com.couchbase.client.CouchbaseConnection.run(CouchbaseConnection.java:288)

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

        Activity

        Hide
        daschl Michael Nitschinger added a comment -

        what do you think is appropriate? The only thing that comes to my mind is checking for every command and throw a InvalidArgumentException?

        Show
        daschl Michael Nitschinger added a comment - what do you think is appropriate? The only thing that comes to my mind is checking for every command and throw a InvalidArgumentException?
        Hide
        perry Perry Krug added a comment -

        I'm not sure what the right approach is (surely Matt can) but the customer indicated that an NPE was always something we shouldn't see

        Show
        perry Perry Krug added a comment - I'm not sure what the right approach is (surely Matt can) but the customer indicated that an NPE was always something we shouldn't see

          People

          • Assignee:
            daschl Michael Nitschinger
            Reporter:
            perry Perry Krug
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:

              Gerrit Reviews

              There are no open Gerrit changes