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

Implement lcb_reconnect() which will reconnect all dead sockets

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: 2.0.0beta2
    • Fix Version/s: None
    • Component/s: library
    • Security Level: Public
    • Labels:
      None

      Description

      This functionality very important for long-living connection handles

        Issue Links

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

          Activity

          Hide
          ingenthr Matt Ingenthron added a comment -

          This issue needs more of a description.

          As discussed with Sergey, it's visible when using libcouchbase in a completely asynchronous app with an external event loop. According to Sergey, this does not affect any of our main use cases with Ruby/PHP/node and rebuilding the connection.

          Show
          ingenthr Matt Ingenthron added a comment - This issue needs more of a description. As discussed with Sergey, it's visible when using libcouchbase in a completely asynchronous app with an external event loop. According to Sergey, this does not affect any of our main use cases with Ruby/PHP/node and rebuilding the connection.
          Hide
          mnunberg Mark Nunberg added a comment -

          I'm closing this as it's possible to attain the same functionality using the lcb_server_versions() command. Higher level SDKs might also wish wrap the versions() command to make it more clear why such an API may be useful.

          Show
          mnunberg Mark Nunberg added a comment - I'm closing this as it's possible to attain the same functionality using the lcb_server_versions() command. Higher level SDKs might also wish wrap the versions() command to make it more clear why such an API may be useful.
          Hide
          ingenthr Matt Ingenthron added a comment -

          I don't exactly see the relationship between the comment and the original need. Can you clarify?

          Show
          ingenthr Matt Ingenthron added a comment - I don't exactly see the relationship between the comment and the original need. Can you clarify?
          Hide
          mnunberg Mark Nunberg added a comment -

          The original request was to provide a routine by which all server sockets can be forcibly connected. Any command which touches all of the servers will make them reconnect the sockets (if they are already dead). Looking at the date of this issue I believe the architecture of the library back then was significantly different from how it is now. The first comment does not really make sense either - at least not in the context of any libcouchbase 2.x released - so I am going by the original bug title.

          Show
          mnunberg Mark Nunberg added a comment - The original request was to provide a routine by which all server sockets can be forcibly connected. Any command which touches all of the servers will make them reconnect the sockets (if they are already dead). Looking at the date of this issue I believe the architecture of the library back then was significantly different from how it is now. The first comment does not really make sense either - at least not in the context of any libcouchbase 2.x released - so I am going by the original bug title.
          Hide
          ingenthr Matt Ingenthron added a comment -

          Sergey: Please clarify and reopen if there's something to be done here. Otherwise, makes sense to me that it be closed.

          Show
          ingenthr Matt Ingenthron added a comment - Sergey: Please clarify and reopen if there's something to be done here. Otherwise, makes sense to me that it be closed.

            People

            • Assignee:
              mnunberg Mark Nunberg
              Reporter:
              avsej Sergey Avseyev
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Gerrit Reviews

                There are no open Gerrit changes