Qpid Proton
  1. Qpid Proton
  2. PROTON-466

Driver - Optimization causes hangs in cleanly closed connections

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: 0.5
    • Fix Version/s: 0.6
    • Component/s: proton-c
    • Labels:
      None

      Description

      Related to https://issues.apache.org/jira/browse/QPID-4853

      Often, when a connection is cleanly closed, the connection-close handshake completes but the connector is never closed. All that is needed is for one more connector_process cycle, but the connector never again appears in the work list.

      There is an optimization in the connector processor that says "if the socket has accepted all of the current data for send, clear the SEL_WR flag". The problem is that when the data-for-send is the last data to be sent on the connection, the connector is never again processed and hangs, using resources and keeping the socket from fully closing.

        Issue Links

          Activity

          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          19h 47m 1 Ted Ross 26/Nov/13 15:54
          Ted Ross made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Hide
          ASF subversion and git services added a comment -

          Commit 1545697 from Ted Ross in branch 'proton/trunk'
          [ https://svn.apache.org/r1545697 ]

          PROTON-466 - driver fix.

          Show
          ASF subversion and git services added a comment - Commit 1545697 from Ted Ross in branch 'proton/trunk' [ https://svn.apache.org/r1545697 ] PROTON-466 - driver fix.
          Ted Ross made changes -
          Link This issue breaks QPID-4853 [ QPID-4853 ]
          Ted Ross made changes -
          Field Original Value New Value
          Attachment PROTON-446.patch [ 12615659 ]
          Hide
          Ted Ross added a comment -

          Patch that fixes the problem.

          Show
          Ted Ross added a comment - Patch that fixes the problem.
          Ted Ross created issue -

            People

            • Assignee:
              Ted Ross
              Reporter:
              Ted Ross
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development