Uploaded image for project: 'Qpid'
  1. Qpid
  2. QPID-4009

Make failover method invocation timeout configurable for 0-8/0-9/0-9-1 amqp clients

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.17
    • Fix Version/s: 0.17
    • Component/s: JMS AMQP 0-x
    • Labels:
      None

      Description

      When connection is lost and failover is configured, on 0-8/0-9/0-9-1 code paths the failover thread is started to restore the connection. The failover thread tries to acquire a failover mutex to re-establish the connection. In some occasions the failover mutex can be hold in JMS methods for a time longer than failover timeout (for example, sending of message with producer flow control on when flow is blocked). As result of it, the failover process is aborted and connection is closed.

      Making the failover method invocation timeout configurable would allows us to work around the issue described above by increasing the failover timeout.

      The proper fix of this issue would be making of failover occurrence absolutely transparent for the client code.

        Activity

        Hide
        alex.rufous Alex Rudyy added a comment -

        Attached a patch making timeout configurable

        Show
        alex.rufous Alex Rudyy added a comment - Attached a patch making timeout configurable
        Hide
        alex.rufous Alex Rudyy added a comment -

        Phil,

        Could you please review the patch attached?

        Show
        alex.rufous Alex Rudyy added a comment - Phil, Could you please review the patch attached?
        Hide
        gemmellr Robbie Gemmell added a comment -

        Patch applied, along with some updates from joint review with Phil.

        Show
        gemmellr Robbie Gemmell added a comment - Patch applied, along with some updates from joint review with Phil.

          People

          • Assignee:
            philharveyonline Philip Harvey
            Reporter:
            alex.rufous Alex Rudyy
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development